-
Notifications
You must be signed in to change notification settings - Fork 363
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(release): publish JSII Python Artifacts to AWS CodeArtifact #3475
feat(release): publish JSII Python Artifacts to AWS CodeArtifact #3475
Conversation
name: "Generate CodeArtifact Token", | ||
run: `echo "TWINE_PASSWORD=$(aws codeartifact get-authorization-token --domain ${domain} --domain-owner ${account} --region ${region} --query authorizationToken --output text)" >> $GITHUB_ENV`, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unfortunately publib does not natively support codeartifact for python like it does for npm. So had to add this step.
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #3475 +/- ##
=======================================
Coverage 96.32% 96.32%
=======================================
Files 191 191
Lines 37407 37485 +78
Branches 3489 3500 +11
=======================================
+ Hits 36032 36108 +76
- Misses 1375 1377 +2 ☔ View full report in Codecov by Sentry. |
src/release/publisher.ts
Outdated
* @param url Of code artifact domain | ||
* @returns domain, account, and region of code artifact domain | ||
*/ | ||
export function awsCodeArtifactInfoFromUrl(url?: string): AwsCodeArtifactInfo { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need for this to be exported.
export function awsCodeArtifactInfoFromUrl(url?: string): AwsCodeArtifactInfo { | |
function awsCodeArtifactInfoFromUrl(url?: string): AwsCodeArtifactInfo { |
src/release/publisher.ts
Outdated
/** | ||
* Info extracted from AWS CodeArtifact URL | ||
*/ | ||
export interface AwsCodeArtifactInfo { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need for this to be exported and part of the public API.
export interface AwsCodeArtifactInfo { | |
interface AwsCodeArtifactInfo { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple of minor changes. Note that removing the export will also require an update the the generated docs again.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice one! This was long overdue
Makes pypi publisher job support targeting AWS CodeArtifact. Re-uses the
CodeArtifactOptions
already added to support publishing of NPM packages to CodeArtifact.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.