Skip to content

Commit 1b7bd1f

Browse files
enkiuszrestyled-commitsarkq
authored
CONTRIBUTING: Add a note discouraging unneccessary CI runs due to the "Update branch" feature being used too often (project-chip#38009)
* CONTRIBUTING: Add a note discouraging unneccessary CI runs due to the "Update branch" feature being used too often Signed-off-by: Maciej Grela <m.grela@samsung.com> * Restyled by prettier-markdown * Update CONTRIBUTING.md Restore proper tip formatting according to GitHub Markdown manual. Co-authored-by: Arkadiusz Bokowy <arkadiusz.bokowy@gmail.com> * Update CONTRIBUTING.md * restyled: restyled release now ships restyler-linux-x86_64 properly now Signed-off-by: Maciej Grela <m.grela@samsung.com> * Restyled by prettier-markdown --------- Signed-off-by: Maciej Grela <m.grela@samsung.com> Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: Arkadiusz Bokowy <arkadiusz.bokowy@gmail.com>
1 parent cb794e0 commit 1b7bd1f

File tree

2 files changed

+9
-5
lines changed

2 files changed

+9
-5
lines changed

.github/workflows/restyled.yml

-5
Original file line numberDiff line numberDiff line change
@@ -15,11 +15,6 @@ jobs:
1515
uses: actions/checkout@v4
1616

1717
- uses: restyled-io/actions/setup@v4
18-
with:
19-
# TODO: Pinned to v0.6.0.2 because the latest release does not have a pre-built
20-
# 'restyler-linux' asset. This broke our workflow as we rely on the pre-built binary.
21-
# Remove this pin once a new release with the 'restyler-linux' asset is available.
22-
tag: 'v0.6.0.2'
2318

2419
- id: restyler
2520
uses: restyled-io/actions/run@v4

CONTRIBUTING.md

+9
Original file line numberDiff line numberDiff line change
@@ -253,6 +253,15 @@ out of convenience.
253253
fixing a typo in an ID still requires some description on how you checked
254254
that the new ID takes effect.
255255

256+
> [!TIP]
257+
>
258+
> When working on a pull request for Matter SDK refrain from using the "Update
259+
> branch" feature in the GitHub UI too often. Updating the PR branch in this way
260+
> triggers the CI workflows cancellation and restart. This feature should be
261+
> used only when a PR has not been worked on for a long time and a lot of
262+
> divergence has accumulated. Your PR branch being out of sync with master is
263+
> not a blocker for merging an approved PR.
264+
256265
### Review Requirements
257266

258267
#### Documentation Best Practices

0 commit comments

Comments
 (0)