Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix errors on Windows #2

Merged
merged 3 commits into from
May 13, 2019
Merged

Fix errors on Windows #2

merged 3 commits into from
May 13, 2019

Conversation

bradrn
Copy link
Contributor

@bradrn bradrn commented Apr 5, 2019

This PR implements some suggestions from #1, in particular:

  • Removing the unused unix dependency to get compilation to work
  • Depending on the open-browser package instead of using custom code, which expands the range of platforms supported (including Windows)

An additional third change has also been implemented, namely rephrasing some info messages; previously some of these messages looked like they were errors, causing the impression that reanimate wasn't working.

This dependency is unused, but its presence causes Windows compilation
to fail.
Adding an ellipsis and rephrasing emphasised that some messages are
info, not error messages. Without this some messages seem like errors
even though they are not.
@lemmih lemmih merged commit acde415 into reanimate:master May 13, 2019
@lemmih
Copy link
Member

lemmih commented May 13, 2019

Great! Thanks!

@bradrn bradrn mentioned this pull request May 13, 2019
lemmih added a commit that referenced this pull request Jun 24, 2020
Fix errors on Windows

Former-commit-id: b2cf3f72f0403ffec17b973e7b4776721dcafd90
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.

None yet

2 participants