Skip to content

Conversation

@justinmk3
Copy link
Contributor

@justinmk3 justinmk3 commented Dec 18, 2024

Problem:
Since 6dcbfc7, the lint-commits job runs whenever the PR description is edited, which then re-runs all the other GHA jobs. This is because it listens to the edited event, which triggers whenever the PR is edited in any way (very noisy).

Solution:
There is no way to avoid rerunning all the CI jobs if they are dependent on lint-commits. Instead, revert 6dcbfc7 and document that re-opening the PR will force CI to re-run, if needed, after fixing a PR title.


  • Treat all work as PUBLIC. Private feature/x branches will not be squash-merged at release time.
  • Your code changes must meet the guidelines in CONTRIBUTING.md.
  • License: I confirm that my contribution is made under the terms of the Apache 2.0 license.

@justinmk3 justinmk3 requested a review from a team as a code owner December 18, 2024 19:55
@justinmk3 justinmk3 changed the title ci: editing PR description re-triggers all GitHub Actions xci: editing PR description re-triggers all GitHub Actions Dec 18, 2024
@justinmk3 justinmk3 changed the title xci: editing PR description re-triggers all GitHub Actions ci: editing PR description re-triggers all GitHub Actions Dec 18, 2024
@justinmk3 justinmk3 closed this Dec 18, 2024
@justinmk3 justinmk3 reopened this Dec 18, 2024
@justinmk3 justinmk3 changed the title ci: editing PR description re-triggers all GitHub Actions xci: editing PR description re-triggers all GitHub Actions Dec 18, 2024
@justinmk3 justinmk3 closed this Dec 18, 2024
@justinmk3 justinmk3 reopened this Dec 18, 2024
* scope: lowercase, <30 chars
* subject: must be <100 chars
* documentation: https://github.com/aws/aws-toolkit-vscode/blob/master/CONTRIBUTING.md#pull-request-title
* Hint: *close and re-open the PR* to re-trigger CI (after fixing the PR title).
Copy link
Contributor Author

Choose a reason for hiding this comment

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

image

@justinmk3 justinmk3 changed the title xci: editing PR description re-triggers all GitHub Actions ci: editing PR description re-triggers all GitHub Actions Dec 18, 2024
@justinmk3 justinmk3 closed this Dec 18, 2024
@justinmk3 justinmk3 reopened this Dec 18, 2024
Problem:
Since 6dcbfc7, the `lint-commits` job runs whenever the PR
*description* is edited, which then re-runs all the other GHA jobs. This
is because it listens to the `edited` event.

Solution:
There is no way to avoid rerunning all the CI jobs if they are dependent
on `lint-commits`. Instead, revert 6dcbfc7 and document that
re-opening the PR will force CI to re-run, if needed, after fixing a PR
title.
@justinmk3 justinmk3 merged commit ab937e4 into aws:master Dec 18, 2024
11 of 17 checks passed
@justinmk3 justinmk3 deleted the fixci branch December 18, 2024 20:37
karanA-aws pushed a commit to karanA-aws/aws-toolkit-vscode that referenced this pull request Jan 17, 2025
Problem:
Since 6dcbfc7, the `lint-commits` job runs whenever the PR
*description* is edited, which then re-runs all the other GHA jobs. This
is because it listens to the `edited` event, which triggers whenever the
PR is edited in any way (very noisy).

Solution:
There is no way to avoid rerunning all the CI jobs if they are dependent
on `lint-commits`. Instead, revert 6dcbfc7 and document that
re-opening the PR will force CI to re-run, if needed, after fixing a PR
title.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant