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

Start testing the gem under Windows #55

Merged
merged 2 commits into from
May 23, 2017

Conversation

deivid-rodriguez
Copy link
Contributor

This is just the old second commit in #46 extracted to a separate PR.

@deivid-rodriguez deivid-rodriguez force-pushed the add_appveyor branch 2 times, most recently from cc182c2 to ad6307d Compare April 21, 2017 17:13
@deivid-rodriguez
Copy link
Contributor Author

Thanks for merging #46, @sickill! 🎉 🎉

I rebased this PR for you in case you want to set this up and make sure everything works on Windows before releasing! 😉

@deivid-rodriguez
Copy link
Contributor Author

@sickill Just some heads up, I got the chance to run the tests under the latest ruby release on Windows, and they all passed! Appveyor would give more confidence since it tests more rubies, but it seems like this gem still works on Windows. :)

@deivid-rodriguez deivid-rodriguez force-pushed the add_appveyor branch 2 times, most recently from 11191e5 to 6954013 Compare May 22, 2017 12:43
@deivid-rodriguez
Copy link
Contributor Author

deivid-rodriguez commented May 22, 2017

I configured this in my fork and it works! https://ci.appveyor.com/project/deivid-rodriguez/rainbow/build/7 :)

@ku1ik ku1ik merged commit dbe286b into ku1ik:master May 23, 2017
@ku1ik
Copy link
Owner

ku1ik commented May 23, 2017

Awesome @deivid-rodriguez, thanks!

@deivid-rodriguez deivid-rodriguez deleted the add_appveyor branch May 24, 2017 19:44
@deivid-rodriguez
Copy link
Contributor Author

@sickill Thanks! You need to link the project to Appveyor though, otherwise this configuration is useless... See here for configuration instructions, just change RuboCop with Rainbow... :)

Also, this commit shouldn't have landed, it was only for debugging. I'll add a PR to revert it!

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