Skip to content

Conversation

Dennis40816
Copy link
Contributor

@Dennis40816 Dennis40816 commented Mar 3, 2025

The functionality introduced in d39d920 was broken due to changes introduced in commit 4a8b8d5, where some occurrences of COMMIT_MSG_LINES were replaced with commit_msg_lines without updating all relevant functions.

This PR fixes the issue by replacing all instances of COMMIT_MSG_LINES with commit_msg_lines.

Copy link
Contributor

@jserv jserv left a comment

Choose a reason for hiding this comment

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

Don't mention the number of pull request. Instead, refer to the short commit hash.

@jserv jserv changed the title Fix the functionality of PR #209 Fix commit hook breakage Mar 3, 2025
@Dennis40816 Dennis40816 force-pushed the unify_commit_msg_lines branch from 7aaeed1 to 82cc680 Compare March 3, 2025 15:47
Copy link
Contributor

@jserv jserv left a comment

Choose a reason for hiding this comment

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

Check the suspicious commit 4a8d8b5 : unknown revision or path not in the working tree.

The functionality introduced in commit d39d920 was broken due to changes
introduced in commit 4a8b8d5, where some occurrences of
'COMMIT_MSG_LINES' were replaced with 'commit_msg_lines' without
updating all relevant functions.

This commit fixes the issue by replacing all instances of
'commit_msg_lines' with 'COMMIT_MSG_LINES', ensuring consistency and
restoring the intended functionality.

Change-Id: I0c93617be83c5f24b1e17678566883c0481ddc48
@Dennis40816 Dennis40816 force-pushed the unify_commit_msg_lines branch from 82cc680 to ff5e2b7 Compare March 3, 2025 15:54
@Dennis40816
Copy link
Contributor Author

Check the suspicious commit 4a8d8b5 : unknown revision or path not in the working tree.

The typo in the commit was fixed.

@jserv jserv merged commit a74d387 into sysprog21:master Mar 3, 2025
1 of 2 checks passed
@jserv
Copy link
Contributor

jserv commented Mar 3, 2025

Thank @Dennis40816 for contributing!

@Dennis40816 Dennis40816 deleted the unify_commit_msg_lines branch March 6, 2025 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants