Skip to content
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

commitlint,FileConventions: partial fix for #120 #128

Merged
merged 4 commits into from
Jul 19, 2023

Conversation

knocte
Copy link
Member

@knocte knocte commented Jul 17, 2023

No description provided.

To make the code more understandable. Also: a micro-
optimization, as there's no need to get nextLine if code
doesn't get inside the next block.
@knocte knocte force-pushed the wip/fixBugsReportedInCommitMsg branch 3 times, most recently from f7cc0ce to 20b939b Compare July 17, 2023 09:29
@knocte knocte force-pushed the wip/fixBugsReportedInCommitMsg branch from 20b939b to 9337018 Compare July 17, 2023 09:34
@knocte knocte force-pushed the wip/fixBugsReportedInCommitMsg branch 2 times, most recently from 9ef1128 to 8829c99 Compare July 19, 2023 09:23
@knocte
Copy link
Member Author

knocte commented Jul 19, 2023

@aarani please review

@knocte knocte force-pushed the wip/fixBugsReportedInCommitMsg branch 3 times, most recently from 3fd01ea to cd7c62e Compare July 19, 2023 13:48
Partial fix for:
#120

Co-authored-by: webwarrior <[email protected]>
@knocte knocte force-pushed the wip/fixBugsReportedInCommitMsg branch from 9811390 to 6cb6d10 Compare July 19, 2023 14:10
@knocte knocte changed the title commitlint(plugins): partial fix for #120 commitlint,FileConventions: partial fix for #120 Jul 19, 2023
@knocte knocte merged commit 1b8f135 into master Jul 19, 2023
knocte added a commit that referenced this pull request Jul 19, 2023
FileConventions,commitlint: partial fix for #120.
knocte added a commit that referenced this pull request Jul 19, 2023
FileConventions,commitlint: partial fix for [1].

[1] #120
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.

1 participant