-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Conversation
Update fork
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
@@ -63,224 +63,226 @@ env: | |||
|
|||
AREA_AFFIXES: > | |||
{ | |||
"prefixes":["@aws-cdk/"] | |||
"prefixes":["@aws-cdk/", "aws-cdk-lib/", "aws-cdk-lib."] |
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.
"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
?
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. |
Closing cuz it will take some additional work to get this up to snuff, and no one is available to do that right now. |
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 inaws-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
@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