Skip to content

Fix for auth workflow due to Click 8.3.0 issue#234

Merged
colinmoynes merged 8 commits intomasterfrom
fix-auth-token-issue
Nov 5, 2025
Merged

Fix for auth workflow due to Click 8.3.0 issue#234
colinmoynes merged 8 commits intomasterfrom
fix-auth-token-issue

Conversation

@colinmoynes
Copy link
Copy Markdown
Contributor

This PR includes a fix for the Click 8.3.0 issue, which was impacting the cloudsmith auth-- token workflow. This restricts to >=8.2.0 and !=8.3.0.

It also includes an update to the actions workflow to help with an issue with zipapp package names.

@colinmoynes colinmoynes requested a review from a team as a code owner November 5, 2025 13:57
Copilot AI review requested due to automatic review settings November 5, 2025 13:57
Copy link
Copy Markdown
Contributor

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 fixes a compatibility issue with Click v8.3.0 and bumps the version to 1.9.1. The breaking change in Click v8.3.0 affected the conversion of Sentinel.UNSET values, which impacted the auth --token workflow.

Key Changes:

  • Excluded Click v8.3.0 from dependencies to prevent breaking changes
  • Fixed indentation bug in config loading logic
  • Enhanced the zipapp push command with explicit naming and versioning

Reviewed Changes

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

Show a summary per file
File Description
setup.py Adds exclusion for Click v8.3.0 to prevent compatibility issues
cloudsmith_cli/data/VERSION Bumps version from 1.9.0 to 1.9.1
cloudsmith_cli/cli/config.py Fixes indentation to ensure config loading only happens when path exists
CODEOWNERS Adds initial CODEOWNERS file with engineering team ownership
CHANGELOG.md Documents the Click v8.3.0 fix in version 1.9.1
.github/workflows/release.yml Adds explicit name and version parameters to zipapp push command
.bumpversion.cfg Updates current version to 1.9.1

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

@colinmoynes colinmoynes merged commit 4de81de into master Nov 5, 2025
13 checks passed
@colinmoynes colinmoynes deleted the fix-auth-token-issue branch November 5, 2025 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants