Skip to content

Commit a5f72d2

Browse files
authored
Remove deprecated connector dependency report (#18889)
* Update action and template * Bump version for testing * Revert change to test comment update * Update action * Update action * Add comment * Test change again * Update comment * Revert change for testing * Add comment * Fix typos
1 parent 418d0ca commit a5f72d2

File tree

2 files changed

+22
-2
lines changed

2 files changed

+22
-2
lines changed

.github/comment_templates/connector_dependency_template.md

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
<!--- this comment is for `report-connectors-dependency.yml` identification, do not remove -->
22

3+
## Affected Connector Report
4+
35
NOTE ⚠️ Changes in this PR affect the following connectors. Make sure to do the following as needed:
46
- Run integration tests
57
- Bump connector version
@@ -17,6 +19,8 @@ NOTE ⚠️ Changes in this PR affect the following connectors. Make sure to do
1719
| --- | :---: | :---: | :---: |
1820
{source_rows}
1921

22+
* See "Actionable Items" below for how to resolve warnings and errors.
23+
2024
</details>
2125

2226
<details {destination_open}>
@@ -30,6 +34,8 @@ NOTE ⚠️ Changes in this PR affect the following connectors. Make sure to do
3034
| --- | :---: | :---: | :---: |
3135
{destination_rows}
3236

37+
* See "Actionable Items" below for how to resolve warnings and errors.
38+
3339
</details>
3440

3541
{others}
@@ -38,7 +44,9 @@ NOTE ⚠️ Changes in this PR affect the following connectors. Make sure to do
3844

3945
<summary>
4046

41-
### Notes
47+
### Actionable Items
48+
49+
(click to expand)
4250

4351
</summary>
4452

.github/workflows/report-connectors-dependency.yml

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,8 @@ jobs:
2525
echo "comment=true" >> $GITHUB_OUTPUT
2626
fi
2727
- name: Find existing comment for connector dependencies
28-
if: steps.dependency_report.outputs.comment == 'true'
28+
# Always run this step because the action may need to
29+
# remove a comment created from a previous commit.
2930
uses: peter-evans/find-comment@v2
3031
id: find_comment
3132
with:
@@ -40,3 +41,14 @@ jobs:
4041
comment-id: ${{ steps.find_comment.outputs.comment-id }}
4142
edit-mode: "replace"
4243
body-file: "comment_body.md"
44+
- name: Remove deprecated report in PR
45+
if: steps.dependency_report.outputs.comment != 'true' && steps.find_comment.outputs.comment-id != ''
46+
uses: peter-evans/create-or-update-comment@v2
47+
with:
48+
issue-number: ${{ github.event.pull_request.number }}
49+
comment-id: ${{ steps.find_comment.outputs.comment-id }}
50+
edit-mode: "replace"
51+
body: |
52+
<!--- this comment is for `report-connectors-dependency.yml` identification, do not remove -->
53+
## Affected Connector Report
54+
The latest commit has removed all connector-related changes. There are no more dependent connectors for this PR.

0 commit comments

Comments
 (0)