Skip to content

CICDL-378: Remove unused use_trusted_publisher input#24

Merged
ahmedabdullajev-pi merged 1 commit into
masterfrom
CICDL-378-remove-use-trusted-publisher
Jun 17, 2026
Merged

CICDL-378: Remove unused use_trusted_publisher input#24
ahmedabdullajev-pi merged 1 commit into
masterfrom
CICDL-378-remove-use-trusted-publisher

Conversation

@ahmedabdullajev-pi

Copy link
Copy Markdown
Contributor

Summary

  • Removes the now-unnecessary use_trusted_publisher: true input from the npm publish workflow call

OIDC Trusted Publishers is the only publish path after pipedrive-actions/github-actions-workflows#4 removed the token-based code path.

⚠️ Merge after pipedrive-actions/github-actions-workflows#4 is merged.

Test plan

  • No functional change — OIDC publish continues to work as before

🤖 Generated with Claude Code

OIDC Trusted Publishers is now the only publish path — the input
is no longer needed after the reusable workflow cleanup.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings June 15, 2026 13:10

Copilot AI left a comment

Copy link
Copy Markdown

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 removes the use_trusted_publisher: true input from the repository’s npm publish workflow call, aligning the workflow invocation with the current OIDC Trusted Publisher publishing path.

Changes:

  • Removed the use_trusted_publisher input from the reusable npm publish workflow invocation.

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

@ahmedabdullajev-pi ahmedabdullajev-pi merged commit ada4c96 into master Jun 17, 2026
2 checks passed
@ahmedabdullajev-pi ahmedabdullajev-pi deleted the CICDL-378-remove-use-trusted-publisher branch June 17, 2026 09:01
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