Skip to content
This repository was archived by the owner on Nov 1, 2024. It is now read-only.

Fix CI errors on master - #73

Merged
sebmartin merged 7 commits into
masterfrom
fix-type-errors
Apr 11, 2019
Merged

sebmartin merged 7 commits into
masterfrom
fix-type-errors

Conversation

@sebmartin

@sebmartin sebmartin commented Apr 10, 2019 •

Copy link
Copy Markdown
Contributor

master has been failing for a while with type errors. This attempts to fix them.

https://travis-ci.org/Shopify/pyoozie/jobs/380235270#L971-L1034

Also mypy stopped supporting python 3.4 about two weeks ago.

@sebmartin
sebmartin force-pushed the fix-type-errors branch 2 times, most recently from ac08145 to 51d01e7 Compare April 10, 2019 16:00
@sebmartin sebmartin changed the title Fix type errors Fix CI errors on master Apr 10, 2019
@sebmartin
sebmartin requested review from cfournie and pior April 10, 2019 16:45
@cfournie cfournie mentioned this pull request Apr 10, 2019

@cfournie cfournie left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What are your thoughts on changing this slightly with #74?

@sebmartin

Copy link
Copy Markdown
Contributor Author

What are your thoughts on changing this slightly with #74?

Yeah I like that better 👍

@sebmartin
sebmartin merged commit e33a323 into master Apr 11, 2019
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