Skip to content

Commit

Permalink
chore: make the service-spec update more user friendly (aws#28441)
Browse files Browse the repository at this point in the history
The current wording made it hard for beginners to understand what the PRs actually do.
This should improve discoverability of L1 changes.

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
  • Loading branch information
mrgrain authored and paulhcsun committed Jan 5, 2024
1 parent aaf781e commit 34e8c14
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions .github/workflows/spec-update.yml
Original file line number Diff line number Diff line change
Expand Up @@ -100,12 +100,12 @@ jobs:
continue-on-error: true
- name: Create PR body file
run: |-
echo 'Update AWS Service Spec packages to latest versions' >> PR.md
echo 'Updates the L1 CloudFormation resource definitions with the latest changes from `@aws-cdk/aws-service-spec`' >> PR.md
- name: Add model changelog to PR body file
if: steps.diff-db.outputs.diff-result
run: |-
echo '' >> PR.md
echo '**@aws-cdk/aws-service-spec changes:**' >> PR.md
echo '**L1 CloudFormation resource definition changes:**' >> PR.md
echo '```' >> PR.md
cat DIFF >> PR.md
echo '```' >> PR.md
Expand Down Expand Up @@ -149,10 +149,10 @@ jobs:
branch: automation/spec-update
author: aws-cdk-automation <[email protected]>
commit-message: |-
feat: update AWS Service Spec
Update AWS Service Spec packages to latest versions
feat: update L1 CloudFormation resource definitions
Updates the L1 CloudFormation resource definitions with the latest changes from `@aws-cdk/aws-service-spec`
# Pull Request details
title: "feat: update AWS Service Spec"
title: "feat: update L1 CloudFormation resource definitions"
body-path: ${{ runner.temp }}/PR.md
labels: contribution/core,dependencies,auto-approve,pr-linter/exempt-integ-test,pr-linter/exempt-readme,pr-linter/exempt-test
team-reviewers: aws-cdk-team
Expand Down

0 comments on commit 34e8c14

Please sign in to comment.