Skip to content

Commit

Permalink
fix duplication of docs issues (#8747)
Browse files Browse the repository at this point in the history
* fix duplication of docs issues

* update conditional to only run on merged PRs
  • Loading branch information
emmyoop authored and QMalcolm committed Oct 6, 2023
1 parent 88dacdf commit e901869
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion .github/workflows/docs-issue.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,14 @@ permissions:

jobs:
open_issues:
if: contains( github.event.pull_request.labels.*.name, 'user docs') && github.event.pull_request.merged == true
# we only want to run this when the PR has been merged or the label in the labeled event is `user docs`. Otherwise it runs the
# risk of duplicaton of issues being created due to merge and label both triggering this workflow to run and neither having
# generating the comment before the other runs. This lives here instead of the shared workflow because this is where we
# decide if it should run or not.
if: |
(github.event.pull_request.merged == true) &&
((github.event.action == 'closed' && contains( github.event.pull_request.labels.*.name, 'user docs')) ||
(github.event.action == 'labeled' && github.event.label.name == 'user docs'))
uses: dbt-labs/actions/.github/workflows/open-issue-in-repo.yml@main
with:
issue_repository: "dbt-labs/docs.getdbt.com"
Expand Down

0 comments on commit e901869

Please sign in to comment.