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: upgrade standard, and fix appveyor config so that it works with newest standard #607

Merged
merged 2 commits into from Sep 8, 2016

Conversation

bcoe
Copy link
Member

@bcoe bcoe commented Aug 26, 2016

No description provided.

@maxrimue
Copy link
Member

@bcoe What do you think about this part of my comment?

What I was wondering btw., should we maybe drop tests for Default Azure too? Since AppVeyor only lists Windows Server 2012 R2 under available operating systems, I think Default Azure just currently means Windows Server 2012 R2, too, which would mean we run the same tests twice.

I saw we have the same approach in yargs-parser's AppVeyor config, too.

@bcoe
Copy link
Member Author

bcoe commented Aug 29, 2016

@maxrimue seems reasonable to also default to Azure.

@bcoe bcoe merged commit c301f42 into master Sep 8, 2016
@bcoe bcoe deleted the upgrade-standard branch September 8, 2016 03:36
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