Add more verbs scraped from another repository #276
Annotations
4 errors
test
The message 1 is invalid:
* The line 3 of the message (line 1 of the body) exceeds the limit of 72 characters. The line contains 371 characters: "We ran into more missing verbs. So I figured instead of adding them a few at a time when we run into issues, let's investigate what we actually have used historically and batch add them. I did the following for our largest repository, this is the only place where we currently enforce git commit message style with this action: https://github.com/mullvad/mullvadvpn-app/.". Please reformat the body so that all the lines fit 72 characters.
* The line 5 of the message (line 3 of the body) exceeds the limit of 72 characters. The line contains 76 characters: "I grabbed all the first words from the subject line of all our ~20k commits:". Please reformat the body so that all the lines fit 72 characters.
* The line 7 of the message (line 5 of the body) exceeds the limit of 72 characters. The line contains 97 characters: "git log --format=%s | awk '{print tolower($1)}' | tr -cd '\\n[:alnum:]._-' | sort -u > first-words". Please reformat the body so that all the lines fit 72 characters.
* The line 10 of the message (line 8 of the body) exceeds the limit of 72 characters. The line contains 74 characters: "I then filtered out the ones not already in `mostFrequentEnglishVerbs.ts`:". Please reformat the body so that all the lines fit 72 characters.
* The line 13 of the message (line 11 of the body) exceeds the limit of 72 characters. The line contains 112 characters: " if ! grep -x \" '$word',\" ~/src/opinionated-commit-message/src/mostFrequentEnglishVerbs.ts > /dev/null; then". Please reformat the body so that all the lines fit 72 characters.
* The line 19 of the message (line 17 of the body) exceeds the limit of 72 characters. The line contains 271 characters: "I then manually started going through the list of words. Many of them are not even verbs, or verbs not in imperative form, since we have not enforced this style before. But many of them are perfectly fine imperative form verbs and I suggest they are added to this action.". Please reformat the body so that all the lines fit 72 characters.
* The line 21 of the message (line 19 of the body) exceeds the limit of 72 characters. The line contains 475 characters: "I stopped at the letter D and decided to submit this PR. That's because I was not sure if you wanted this addition or not, and figured I should reach out for feedback before spending more time. I'll likely continue the filtering anyway, and add it to a local `path-to-additional-verbs` in case you don't want them upstream. But having them upstream would be way more handy, since then we can easily run this on multiple repositories without having to duplicate the verb list.". Please reformat the body so that all the lines fit 72 characters.
* The line 23 of the message (line 21 of the body) exceeds the limit of 72 characters. The line contains 145 characters: "Semi-related question: Does it make sense to separate \"Verbs specific to programming\"? I personally think that distinction is not very important.". Please reformat the body so that all the lines fit 72 characters.
The original message was:
Add more verbs from A through D
We ran into more missing verbs. So I figured instead of adding them a few at a time when we run into issues, let's investigate what we actually have used historically and batch add them. I did the following for our largest repository, this is the only place where we currently enforce git commit message style with this action: https://github.com/mullvad/mullvadvpn-app/.
I grabbed all the first words from the subject line of all our ~20k commits:
```bash
git log --format=%s | awk '{print tolower($1)}' | tr -cd '\n[:alnum:]._-' | sort -u > first-words
```
I then filtered out the ones not already in `mostFrequentEnglishVerbs.ts`:
```bash
for word in $(cat first-words); do
if ! grep -x " '$word'," ~/src/opinionated-commit-message/src/mostFrequentEnglishVerbs.ts > /dev/null; then
echo "$word";
fi;
done
```
I then manually sta
|
test-allow-one-liners
The message 1 is invalid:
* The line 3 of the message (line 1 of the body) exceeds the limit of 72 characters. The line contains 371 characters: "We ran into more missing verbs. So I figured instead of adding them a few at a time when we run into issues, let's investigate what we actually have used historically and batch add them. I did the following for our largest repository, this is the only place where we currently enforce git commit message style with this action: https://github.com/mullvad/mullvadvpn-app/.". Please reformat the body so that all the lines fit 72 characters.
* The line 5 of the message (line 3 of the body) exceeds the limit of 72 characters. The line contains 76 characters: "I grabbed all the first words from the subject line of all our ~20k commits:". Please reformat the body so that all the lines fit 72 characters.
* The line 7 of the message (line 5 of the body) exceeds the limit of 72 characters. The line contains 97 characters: "git log --format=%s | awk '{print tolower($1)}' | tr -cd '\\n[:alnum:]._-' | sort -u > first-words". Please reformat the body so that all the lines fit 72 characters.
* The line 10 of the message (line 8 of the body) exceeds the limit of 72 characters. The line contains 74 characters: "I then filtered out the ones not already in `mostFrequentEnglishVerbs.ts`:". Please reformat the body so that all the lines fit 72 characters.
* The line 13 of the message (line 11 of the body) exceeds the limit of 72 characters. The line contains 112 characters: " if ! grep -x \" '$word',\" ~/src/opinionated-commit-message/src/mostFrequentEnglishVerbs.ts > /dev/null; then". Please reformat the body so that all the lines fit 72 characters.
* The line 19 of the message (line 17 of the body) exceeds the limit of 72 characters. The line contains 271 characters: "I then manually started going through the list of words. Many of them are not even verbs, or verbs not in imperative form, since we have not enforced this style before. But many of them are perfectly fine imperative form verbs and I suggest they are added to this action.". Please reformat the body so that all the lines fit 72 characters.
* The line 21 of the message (line 19 of the body) exceeds the limit of 72 characters. The line contains 475 characters: "I stopped at the letter D and decided to submit this PR. That's because I was not sure if you wanted this addition or not, and figured I should reach out for feedback before spending more time. I'll likely continue the filtering anyway, and add it to a local `path-to-additional-verbs` in case you don't want them upstream. But having them upstream would be way more handy, since then we can easily run this on multiple repositories without having to duplicate the verb list.". Please reformat the body so that all the lines fit 72 characters.
* The line 23 of the message (line 21 of the body) exceeds the limit of 72 characters. The line contains 145 characters: "Semi-related question: Does it make sense to separate \"Verbs specific to programming\"? I personally think that distinction is not very important.". Please reformat the body so that all the lines fit 72 characters.
The original message was:
Add more verbs from A through D
We ran into more missing verbs. So I figured instead of adding them a few at a time when we run into issues, let's investigate what we actually have used historically and batch add them. I did the following for our largest repository, this is the only place where we currently enforce git commit message style with this action: https://github.com/mullvad/mullvadvpn-app/.
I grabbed all the first words from the subject line of all our ~20k commits:
```bash
git log --format=%s | awk '{print tolower($1)}' | tr -cd '\n[:alnum:]._-' | sort -u > first-words
```
I then filtered out the ones not already in `mostFrequentEnglishVerbs.ts`:
```bash
for word in $(cat first-words); do
if ! grep -x " '$word'," ~/src/opinionated-commit-message/src/mostFrequentEnglishVerbs.ts > /dev/null; then
echo "$word";
fi;
done
```
I then manually sta
|
build
The message 1 is invalid:
* The subject must start with a capitalized word, but the current first word is: "change". Please capitalize to: "Change".
The original message was:
change SomeClass to OtherClass
SomeClass with OtherClass
|
build
The message 1 is invalid:
* The subject must start with a capitalized word, but the current first word is: "change". Please capitalize to: "Change".
The original message was:
change SomeClass to OtherClass
Do something
The message 2 is invalid:
* The first word of the subject ("Change") must not match the first word of the body. Please make the body more informative by adding more information instead of repeating the subject. For example, start by explaining the problem that this change is intended to solve or what was previously missing (e.g., "Previously, ....").
The original message was:
Change other subject
Change body
|