Skip to content

Conversation

@silug
Copy link
Collaborator

@silug silug commented Jan 26, 2026

Also test with Ruby 4.0

michael-riddle
michael-riddle previously approved these changes Jan 26, 2026
@silug silug changed the title chore(dependencies): Support compliance_engine 0.2.x chore(deps): Support compliance_engine 0.2.x Jan 26, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR broadens the supported compliance_engine versions, updates dependencies, bumps the gem version, and extends CI to test against Ruby 4.0.

Changes:

  • Loosen compliance_engine runtime dependency to support 0.1.x–0.2.x and add an explicit json-schema dependency in the gemspec.
  • Bump the library version to 0.4.1 and document the release in CHANGELOG.md.
  • Update the GitHub Actions workflow to run style and spec tests on Ruby 4.0 and clarify the 3.2 job comment.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
scelint.gemspec Expands compliance_engine version range to < 0.3.0 and adds json-schema ~> 5.0 as a runtime dependency, enabling compatibility with newer compliance engine releases.
lib/scelint/version.rb Increments the gem’s version constant to 0.4.1 to reflect the new release.
CHANGELOG.md Adds a new 0.4.1 entry describing compliance_engine 0.2.x support and Ruby 4.0 testing.
.github/workflows/pr_tests.yml Extends CI to Ruby 4.0 (style and spec jobs) and updates the Ruby 3.2 matrix entry comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@silug silug merged commit 3c64297 into simp:master Jan 26, 2026
6 checks passed
@github-project-automation github-project-automation bot moved this from New to Done in Org Triage Jan 26, 2026
@silug silug deleted the compliance_engine-0.2 branch January 26, 2026 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants