Skip to content

appveyor.yml - correct case in paths, use trunk #714

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

Merged
merged 2 commits into from
May 2, 2019

Conversation

MSP-Greg
Copy link
Contributor

@MSP-Greg MSP-Greg commented Apr 30, 2019

This can cause very odd issues, seemingly coming out of nowhere. I had the mistake in some repos also...

2nd commit uses trunk.

@MSP-Greg MSP-Greg changed the title appveyor.yml - correct case in paths appveyor.yml - correct case in paths, use 2.6 & trunk Apr 30, 2019
@MSP-Greg
Copy link
Contributor Author

MSP-Greg commented Apr 30, 2019

Test times should be much faster, as Appveyor does not set a default clone depth, so the git clone was grabbing everything. I set it to match Travis, which is 50.

Appveyor docs state that all versions of Ruby exist in all VS images, but there may be an issue with the default (2015?). The 2017 image does have 2.6. I'll raise an issue with Appveyor about 2.6.

Copy link
Member

@aycabta aycabta left a comment

Choose a reason for hiding this comment

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

Please don't use 2.6 on AppVeyor yet.

@MSP-Greg
Copy link
Contributor Author

MSP-Greg commented May 1, 2019

Done. Forgive me, might I inquire why?

@MSP-Greg MSP-Greg changed the title appveyor.yml - correct case in paths, use 2.6 & trunk appveyor.yml - correct case in paths, use trunk May 1, 2019
@aycabta
Copy link
Member

aycabta commented May 2, 2019

Unfortunately, AppVeyor doesn't support 2.6 yet...

@aycabta aycabta merged commit b94d0d9 into ruby:master May 2, 2019
@MSP-Greg
Copy link
Contributor Author

MSP-Greg commented May 2, 2019

@aycabta

Appveyor does support 2.6. It is an Appveyor issue, but the problem is only with older (?) VS images. The image can be set in the UI, or in the yaml. I added a newer image to the yaml, and it works fine (2.6.1)

See
https://ci.appveyor.com/project/ruby/rdoc/builds/24219700/job/fc8725pb2fc6hv2m
and
https://ci.appveyor.com/project/ruby/rdoc/builds/24219700/job/u95t56kk7aksf5y1#L98

They will be updating 2.4, 2.5, & 2.6 sometime in the near future.

JFYI, you may (from time to time) find CI Ruby version summaries I run on CRON sessions helpful:

https://ci.appveyor.com/project/MSP-Greg/appveyor-ruby

https://travis-ci.org/MSP-Greg/travis-ruby

@MSP-Greg MSP-Greg deleted the appveyor-case branch May 3, 2019 04:04
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