-
-
Notifications
You must be signed in to change notification settings - Fork 60
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
Remove CI Rails versions #172
Conversation
WalkthroughThe pull request simplifies the GitHub Actions workflow by removing the job definitions for rails_4_2 and rails_5_2 in the .github/workflows/ruby.yml file. Only the rails_3_2 job remains active, which sets up Ruby version 2.3 and runs tests via Rake. This change reduces complexity by eliminating duplicate testing scenarios for different Ruby versions. Changes
Sequence Diagram(s)sequenceDiagram
participant G as GitHub Actions
participant J as rails_3_2 Job
participant R as Ruby Environment
G->>J: Trigger workflow event
J->>R: Set up Ruby (version 2.3)
R-->>J: Environment ready
J->>G: Execute tests via Rake
G-->>J: Report test results
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
⏰ Context from checks skipped due to timeout of 90000ms (1)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
@grantspeelman We could start from here. Remove all versions except 3.2 (some of them fail on master sometimes) |
I haved removed all of them. Will add them back once everything is merged. |
Summary by CodeRabbit