Skip to content
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

chore: change SSM timestamp name to cli-npm #185

Merged
merged 2 commits into from
Mar 3, 2025

Conversation

rix0rrr
Copy link
Contributor

@rix0rrr rix0rrr commented Mar 3, 2025

We used to publish under the key name cli, but the library publishing process is writing timestamps under npm, pypi, etc.

Having the keys

  • npm
  • pypi
  • nuget
  • ...
  • cli

Was confusing, so we're changing it to cli-npm instead.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

We used to publish under the key name `cli`, but the library publishing
process is writing timestamps under `npm`, `pypi`, etc.

Having the keys

* `npm`
* `pypi`
* `nuget`
* ...
* `cli`

Was confusing, so we're changing it to `cli-npm` instead.
@rix0rrr rix0rrr requested a review from a team March 3, 2025 13:12
@aws-cdk-automation aws-cdk-automation requested a review from a team March 3, 2025 13:12
@github-actions github-actions bot added the p2 label Mar 3, 2025
@@ -143,15 +143,13 @@ jobs:
- init-typescript-lib
- tool-integrations
integ:
needs:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Got some locally linked projen stuff? Should be fine with self-mutate I guess.

@rix0rrr rix0rrr added the pr/exempt-integ-test Skips the integ test steps if set. label Mar 3, 2025
@codecov-commenter
Copy link

codecov-commenter commented Mar 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.75%. Comparing base (12299ff) to head (b4376d3).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #185      +/-   ##
==========================================
- Coverage   84.84%   84.75%   -0.09%     
==========================================
  Files         205      205              
  Lines       35648    35648              
  Branches     4617     4607      -10     
==========================================
- Hits        30245    30213      -32     
- Misses       5245     5277      +32     
  Partials      158      158              
Flag Coverage Δ
suite.unit 84.75% <ø> (-0.09%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: github-actions <[email protected]>
@aws-cdk-automation aws-cdk-automation added this pull request to the merge queue Mar 3, 2025
Merged via the queue into main with commit 806d851 Mar 3, 2025
11 checks passed
@aws-cdk-automation aws-cdk-automation deleted the huijbers/change-ssm-key branch March 3, 2025 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p2 pr/exempt-integ-test Skips the integ test steps if set.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants