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

terse not tense #39

Closed
ghostsquad opened this issue Feb 25, 2020 · 6 comments
Closed

terse not tense #39

ghostsquad opened this issue Feb 25, 2020 · 6 comments
Labels

Comments

@ghostsquad
Copy link

I noticed that gacp is asking me to write a short, imperative tense description. I think the word to be used is terse.

@vivaxy
Copy link
Owner

vivaxy commented Feb 25, 2020

Thanks for pointing out.

@vivaxy vivaxy closed this as completed in 2bd14f5 Feb 25, 2020
@ghostsquad
Copy link
Author

you are right! my mistake. Maybe you could just remove the word altogether? I think saying imperative description still works well.

@vivaxy
Copy link
Owner

vivaxy commented Feb 26, 2020

you are right! my mistake. Maybe you could just remove the word altogether? I think saying imperative description still works well.

To be discussed.

@vivaxy vivaxy reopened this Feb 26, 2020
@vivaxy vivaxy closed this as completed Jun 23, 2020
@vivaxy
Copy link
Owner

vivaxy commented Jun 23, 2020

It's a common word when talking about the commit messages in commitlint, I think it would be better if we keep it that way.

@vivaxy vivaxy reopened this Jun 23, 2020
@ghostsquad
Copy link
Author

Yes, I think a revert is needed here. I apologize for the churn.

@vivaxy vivaxy closed this as completed Jun 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants