Skip to content

Improve documentation for role-to-assume#381

Merged
hsbt merged 2 commits intomainfrom
doc
Mar 24, 2026
Merged

Improve documentation for role-to-assume#381
hsbt merged 2 commits intomainfrom
doc

Conversation

@hsbt
Copy link
Copy Markdown
Member

@hsbt hsbt commented Mar 24, 2026

Clarify that role-to-assume@ expects an OIDC API Key Role token string (rg_oidc_akr_...`) rather than a numeric ID, add documentation for Trusted Publisher as the recommended approach, and update the table of contents to reflect the new structure.

Fixes #161

…ction

Clarify that role-to-assume expects an OIDC API Key Role token string
(rg_oidc_akr_...) rather than a numeric ID, add documentation for
Trusted Publisher as the recommended approach, and update the table
of contents to reflect the new structure.

Fixes #161

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings March 24, 2026 09:32
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

Updates the README documentation to clarify how to configure rubygems/configure-rubygems-credentials for publishing, especially around role-to-assume, and to recommend RubyGems Trusted Publishing as the primary approach.

Changes:

  • Reworks the Usage section into three configuration paths: Trusted Publisher, OIDC API Key Role, and static API token.
  • Clarifies that role-to-assume is an OIDC API Key Role token string (rg_oidc_akr_...) rather than a numeric ID.
  • Updates the Table of Contents to match the new structure.

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

Comment thread README.md
…tions

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@hsbt hsbt merged commit 2d96c9b into main Mar 24, 2026
12 checks passed
@hsbt hsbt deleted the doc branch March 24, 2026 09:47
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.

Documentation for role-to-assume

2 participants