Skip to content

feat(dev-infra): save invalid commit message attempts to be restored on next commit attempt #38304

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed

Conversation

josephperrott
Copy link
Member

When a commit message fails validation, rather than throwing out the commit message entirely
the commit message is saved into a draft file and restored on the next commit attempt.

@josephperrott josephperrott added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release area: dev-infra Issues related to Angular's own dev infra (build, test, CI, releasing) labels Jul 30, 2020
@ngbot ngbot bot modified the milestone: needsTriage Jul 30, 2020
@josephperrott
Copy link
Member Author

Setting @alxhub as a reviewer as this feature was at his request. Can you confirm this is what you were looking for?

Copy link
Member

@gkalpak gkalpak left a comment

Choose a reason for hiding this comment

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

🎉

@josephperrott josephperrott force-pushed the commit-message-resume branch 2 times, most recently from 7e4983e to f024f67 Compare July 31, 2020 16:51
Copy link
Member

@devversion devversion left a comment

Choose a reason for hiding this comment

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

Nice! A few minor comments.

@josephperrott josephperrott requested review from gkalpak and removed request for alxhub August 11, 2020 16:01
@josephperrott josephperrott added the action: merge The PR is ready for merge by the caretaker label Aug 11, 2020
@josephperrott josephperrott force-pushed the commit-message-resume branch from 6fbbbf1 to 36c1bd2 Compare August 12, 2020 15:33
…on next commit attempt

When a commit message fails validation, rather than throwing out the commit message entirely
the commit message is saved into a draft file and restored on the next commit attempt.
@josephperrott josephperrott force-pushed the commit-message-resume branch from 36c1bd2 to c09377a Compare August 12, 2020 23:53
@josephperrott josephperrott removed the action: review The PR is still awaiting reviews from at least one requested reviewer label Aug 13, 2020
atscott pushed a commit that referenced this pull request Aug 13, 2020
…on next commit attempt (#38304)

When a commit message fails validation, rather than throwing out the commit message entirely
the commit message is saved into a draft file and restored on the next commit attempt.

PR Close #38304
@atscott atscott closed this in f4ced74 Aug 13, 2020
@josephperrott josephperrott deleted the commit-message-resume branch August 13, 2020 19:53
profanis pushed a commit to profanis/angular that referenced this pull request Sep 5, 2020
…on next commit attempt (angular#38304)

When a commit message fails validation, rather than throwing out the commit message entirely
the commit message is saved into a draft file and restored on the next commit attempt.

PR Close angular#38304
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker area: dev-infra Issues related to Angular's own dev infra (build, test, CI, releasing) cla: yes target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants