Skip to content

Add tests and CI, and remove rubyci-screenshot - #79

Merged
hsbt merged 3 commits into
masterfrom
claude/add-tests-and-ci-f52ca8
Oct 14, 2026
Merged

hsbt merged 3 commits into
masterfrom
claude/add-tests-and-ci-f52ca8

Conversation

@hsbt

@hsbt hsbt commented Oct 14, 2026

Copy link
Copy Markdown
Member

Dependabot bumps had no CI to check them, and the rubyzip 3.4.0 bump in #76 broke rubyci-screenshot. rubyzip 3.4.0 requires Ruby 3.0, while the workflow ran on Ruby 2.7 and selenium-webdriver 3.142.6 does not work on Ruby 3.

The screenshot had not succeeded once in the last year, so I removed it instead of fixing it. This also drops selenium-webdriver and rubyzip from the Gemfile.

I added tests that run the report extraction and timestamp filtering of rubyci-alerts.rb against fixtures trimmed from rubyci.org's servers.json and reports.json, and a test workflow that runs them on push and pull requests. rubyci-alerts now runs on Ruby 4.0, the same version as the tests, since Ruby 2.7 is EOL.

Generated with Claude Code

hsbt and others added 3 commits October 14, 2026 09:01
It has not succeeded once in the last year, and the rubyzip 3.4.0 bump left no Ruby that can run it, because rubyzip now requires Ruby 3.0 while selenium-webdriver 3.142.6 does not work on Ruby 3.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Ruby 2.7 is EOL.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Dependency bumps from dependabot had nothing to check them. The fixtures are records trimmed from rubyci.org's servers.json and reports.json.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@hsbt
hsbt merged commit 9ae8424 into master Oct 14, 2026
1 check passed
@hsbt
hsbt deleted the claude/add-tests-and-ci-f52ca8 branch October 14, 2026 00:17
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.

1 participant