Skip to content

Fix master#332

Merged
WhyNotHugo merged 4 commits into
masterfrom
fix-master
Nov 9, 2018
Merged

Fix master#332
WhyNotHugo merged 4 commits into
masterfrom
fix-master

Conversation

@WhyNotHugo

Copy link
Copy Markdown
Member

Clean up a flake8 failure. No idea how this got into master.

Also update the changelog, since I had this change just locally, for some reason 🤷‍♂️

@penguineer

Copy link
Copy Markdown
Contributor

Will this fix the test problems in #329?

@WhyNotHugo

Copy link
Copy Markdown
Member Author

That's my intention, but python is failing to install on macOS, and I can't quite figure out how to get around it.

To make things worse, Jenkins macOS builds take AGES, so this is quite time-consuming to debug. :(

But yes, master is currently broken, and the failures on #329 are because of these failures.

@penguineer penguineer mentioned this pull request Oct 25, 2018
@penguineer

Copy link
Copy Markdown
Contributor

Is there a way to help with this issue?

Since this has been broken for a while, and it seems non-trivial to fix
(or rather, lack of time/interest), allow CI to fail for macOS so we can
unfreeze master.
@penguineer

Copy link
Copy Markdown
Contributor

7425dd7 actually made me lol :D

@WhyNotHugo WhyNotHugo merged commit a9e95a1 into master Nov 9, 2018
@WhyNotHugo WhyNotHugo deleted the fix-master branch November 9, 2018 14:38
@WhyNotHugo

Copy link
Copy Markdown
Member Author

Opened #335 to track macOS failures separately.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants