-
-
Notifications
You must be signed in to change notification settings - Fork 28
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
Github workflow for a gem #80
base: main
Are you sure you want to change the base?
Conversation
- Basically from this sample but changing the Rubyies https://github.com/ruby/setup-ruby#matrix-of-ruby-versions - Matrix over ubuntu-latest and macos-latest ruby version
Hey, sorry for the delay. https://github.com/ManageIQ/vmware_web_service/actions/runs/1775391270/workflow looks like an example of using a Ruby + Rails matrix. I'll enable GitHub Actions now. |
RE: delay - no worries, with open source, patience is a virtue. I'm on UK time BTW. TEST_RAILS_VERSIONI'm trying to understand where Rails version comes into it? The CI doesn't install Rails (see image below). Is the idea that the environment TEST_RAILS_VERSION is used somewhere in the code? I looked for the environment in your Gem, Capybara, and active_support (I won't pretend I know what I'm doing in regards to Rails but I thought I should at least try). Within active_support I found a method Ubuntu and Mac OSThe code in the PR varied the test run over Ubuntu and Mac OS I thought that might be useful because I run Capybara locally, on. a Macbook, and in a CI. I'm assuming you don't want this? |
Ah sorry I saw just the matrix and thought this was being handled by GitHub. https://github.com/ManageIQ/vmware_web_service/blob/b47471f9be1eed421cfd1613e321dea71755a60b/Gemfile switches on an ENV variable for the ActiveSupport version (we'll use Rails) defined in Gemfile. Gemfile isn't used in published gems, so that gives us a place to store the versions without impacting users. https://github.com/ruby/setup-ruby#matrix-of-gemfiles is another option. You can see that they use the gemfile matrix variable to find a gemfile then set BUNDLE_GEMFILE with that, which will tell bundler to look at different files per run.
I have no problem with it, but if we're limited to 2 dimensional matrices or if it's going to blow up the number of runs, I'd say we leave it out. I don't think it is going to provide a huge amount of value as Ruby is interpreted, not compiled, so we'd only be looking at differences between compiled dependencies, which should be handling their own compatibility in their testing. |
👋 Hello Caleb RequirementsMatrix that iterates over:
RE: TEST_RAILS_VERSION: I must admit, I'm not sure how TEST_RAILS_VERSION is used unless we call VMWare during the CI run?? But my confusion isn't that important as long as I know what you require, ☝️, since, unlike me, you know what you're talking about. RE: Ubuntu and Mac OS: I'll drop that part of the code. We will use ubuntu_latest You don't want anything else in the steps other than rspec running? |
#77
Basically from this sample but changing the Rubyies https://github.com/ruby/setup-ruby#matrix-of-ruby-versions
Matrix over ubuntu-latest and macos-latest ruby version