Skip to content

Commit 95dcbe6

Browse files
kopporsubhramit
andauthored
Refine no-force-push (#12993)
* Refine no-force-push * Tone down, tone up. --------- Co-authored-by: Subhramit Basu <subhramit.bb@live.in>
1 parent 2616b5e commit 95dcbe6

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

.github/ghprcomment.yml

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -88,10 +88,13 @@
8888
- jobName: no-force-push
8989
always: true
9090
message: >
91-
Do not force-push!
92-
Force pushing is a very bad practice when working together on a project (mainly because it is [not supported well by GitHub itself](https://github.com/orgs/community/discussions/3478)).
91+
Hey, we noticed that you **force-pushed** your changes.
92+
Force pushing is a bad practice when working together on a project (mainly because it is [not supported well by GitHub itself](https://github.com/orgs/community/discussions/3478)).
9393
Commits are lost and comments on commits lose their context, thus making it harder to review changes.
9494
At the end, all commits will be [squashed](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/incorporating-changes-from-a-pull-request/about-pull-request-merges#squash-and-merge-your-commits) anyway before being merged into the `main` branch.
95+
96+
97+
In future, **please avoid that**. For now, you can continue working.
9598
- jobName: 'Conflicts with target branch'
9699
message: >
97100
Your pull request conflicts with the target branch.

0 commit comments

Comments
 (0)