Skip to content

v2#16

Merged
enjaku4 merged 12 commits intomasterfrom
v2
Mar 31, 2026
Merged

v2#16
enjaku4 merged 12 commits intomasterfrom
v2

Conversation

@enjaku4
Copy link
Copy Markdown
Owner

@enjaku4 enjaku4 commented Mar 12, 2026

No description provided.

Copy link
Copy Markdown

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 bumps Kreds to v2.0.0 and aligns the gem (and tooling) with newer supported Ruby/Rails versions, including a small internal adjustment to how credentials are surfaced and how var fallback presence is checked.

Changes:

  • Bump gem version to 2.0.0 and document the breaking support changes in CHANGELOG.md.
  • Drop support for Ruby 3.2 and Rails 7.1 (gemspec constraints, default Gemfile Rails version, RuboCop targets, CI matrix).
  • Adjust credentials inspection (Kreds.show) and tweak env-var fallback guard logic.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
lib/kreds/version.rb Major version bump to 2.0.0.
lib/kreds/show.rb Changes how credentials are converted to a hash for Kreds.show.
lib/kreds/fetch.rb Uses nil? (vs blank?) to decide whether to attempt env var fallback after input processing.
kreds.gemspec Raises minimum Ruby to 3.3 and minimum Rails to 7.2.
Gemfile Updates default Rails version to ~> 7.2.
CHANGELOG.md Adds v2.0.0 notes about dropped Ruby/Rails support.
.rubocop.yml Updates target Ruby/Rails versions for linting.
.github/workflows/ci.yml Updates lint Ruby version and CI test matrix to match the new support policy.

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

Comment thread .github/workflows/ci.yml
Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 12, 2026

@enjaku4 I've opened a new pull request, #17, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI and others added 2 commits March 12, 2026 21:08
Co-authored-by: enjaku4 <32835381+enjaku4@users.noreply.github.com>
Copy link
Copy Markdown

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

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.


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

Comment thread lib/kreds/show.rb
Copy link
Copy Markdown

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

Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.


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

Comment thread CHANGELOG.md
Comment thread spec/kreds_spec.rb
Copy link
Copy Markdown

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

Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.


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

Comment thread lib/kreds/show.rb Outdated
Copy link
Copy Markdown

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

Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.


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

Comment thread README.md
@enjaku4 enjaku4 merged commit d1cf66f into master Mar 31, 2026
15 checks passed
@enjaku4 enjaku4 deleted the v2 branch March 31, 2026 16:25
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.

3 participants