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: update area labels to match repository restructure #27232

Closed
wants to merge 13 commits into from

Conversation

scanlonp
Copy link
Contributor

@scanlonp scanlonp commented Sep 21, 2023

Draft: This is something I would like to do in the future. However, possible dependencies on our current labels (metrics, actions, etc) makes this something that does not need to be done at the moment.

All of our "area" labels currently are prefixed with @aws-cdk. Since the repository was restructured to have stable modules live in aws-cdk-lib and alpha modules live in @aws-cdk, our labels should reflect that.

Additionally, updated several labels for modules that have been graduated from alpha to stable.

Dependent on #27231 and

  • running the associated action before merging this workflow update.
  • labels for @aws-cdk/aws-kinesis-firehose-destinations-alpha, @aws-cdk/aws-scheduler-alpha, and @aws-cdk/aws-scheduler-targets-alpha need to be created.

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

@scanlonp scanlonp added the pr/do-not-merge This PR should not be merged at this time. label Sep 21, 2023
@aws-cdk-automation aws-cdk-automation requested a review from a team September 21, 2023 01:21
@github-actions github-actions bot added the p2 label Sep 21, 2023
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Sep 21, 2023
@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildv2Project1C6BFA3F-wQm2hXv2jqQv
  • Commit ID: 44663b1
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@aws-cdk-automation aws-cdk-automation added the pr/needs-maintainer-review This PR needs a review from a Core Team Member label Sep 21, 2023
@@ -63,224 +63,226 @@ env:

AREA_AFFIXES: >
{
"prefixes":["@aws-cdk/"]
"prefixes":["@aws-cdk/", "aws-cdk-lib/", "aws-cdk-lib."]
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"prefixes":["@aws-cdk/", "aws-cdk-lib/", "aws-cdk-lib."]
"prefixes":["@aws-cdk/", "aws-cdk-lib/", "aws-cdk-lib."]

Can we turn 'em in to module/xxx or service/xxx ?

@scanlonp
Copy link
Contributor Author

scanlonp commented Dec 6, 2023

I like the relative path much more than service or module. Module is better than service since not all of the packages are services in and of themselves.

Module provides little to no additional value. @aws-cdk and aws-cdk-lib give a path to the package, and makes an easier distinction for alpha vs stable modules.

@scanlonp scanlonp marked this pull request as draft December 6, 2023 23:37
@aws-cdk-automation aws-cdk-automation removed the pr/needs-maintainer-review This PR needs a review from a Core Team Member label Dec 6, 2023
@kaizencc
Copy link
Contributor

Closing cuz it will take some additional work to get this up to snuff, and no one is available to do that right now.

@kaizencc kaizencc closed this Dec 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution/core This is a PR that came from AWS. p2 pr/do-not-merge This PR should not be merged at this time.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants