Skip to content
This repository was archived by the owner on Jun 9, 2020. It is now read-only.

Add .travis.yml #155

Merged
merged 1 commit into from
Mar 29, 2019
Merged

Add .travis.yml #155

merged 1 commit into from
Mar 29, 2019

Conversation

jayvdb
Copy link
Contributor

@jayvdb jayvdb commented Mar 28, 2019

No description provided.

@jayvdb
Copy link
Contributor Author

jayvdb commented Mar 28, 2019

Test errors noted at #153 and #154

@ricea
Copy link
Contributor

ricea commented Mar 28, 2019

If I understand correctly, this will consistently fail until #153 and #154 are fixed. Maybe we should delay introducing it?

@jayvdb
Copy link
Contributor Author

jayvdb commented Mar 28, 2019

Consistently failing is fine. At least people can see and start working on the problems.

The Python 2 problems are obviously highest priority, so I expect that will be solved quickly.

The SyntaxError's on Python 3 are easy to fix - then we can assess whether Python 3 support is feasible in the short term -- if not, the Python 3 job can be made 'non-voting'.

@ricea
Copy link
Contributor

ricea commented Mar 28, 2019

I'm checking if we can actually use Travis on this repository. May take a day or two.

@jayvdb
Copy link
Contributor Author

jayvdb commented Mar 28, 2019

ok, thanks Adam.

@ricea
Copy link
Contributor

ricea commented Mar 29, 2019

Maybe okay now? Going to try it and see if it works.

@ricea ricea merged commit b79b28f into googlearchive:master Mar 29, 2019
@ricea
Copy link
Contributor

ricea commented Mar 29, 2019

Yes, that seems to work.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants