-
Notifications
You must be signed in to change notification settings - Fork 12
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
Incrementalify #20
Incrementalify #20
Conversation
Also updates several items in the pom
c9e200a
to
07892be
Compare
I regularly use the label verifier plugin in my installation to assure that I've correctly labeled agents with the command line git version that they are running. It helps me run targeted tests on specific versions of command line git. Pull requests to be merged include: * jenkinsci/label-verifier-plugin#18 * jenkinsci/label-verifier-plugin#19 * jenkinsci/label-verifier-plugin#20 * jenkinsci/label-verifier-plugin#21 * jenkinsci/label-verifier-plugin#22 * jenkinsci/label-verifier-plugin#23 https://plugins.jenkins.io/label-verifier/ shows that the plugin is up for adoption.
<packaging>hpi</packaging> | ||
<name>Jenkins Label Verifier plugin</name> | ||
<url>https://github.com/jenkinsci/label-verifier-plugin</url> | ||
<name>Label verifier plugin</name> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why the change from title case?
Also https://www.jenkins.io/doc/developer/publishing/style-guides/#name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wasn't aware that title case was a convention. I'll happily adjust that and remove the redundant "plugin" from the end of the name.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wasn't aware that title case was a convention
I don't think it's documented or required, but it's done fairly consistently. A quick and dirty check indicates that 90+% of plugin titles with 2+ words are title case.
Incremtnality
Allow incrementals for testing
Testing done
Automated tests pass
Submitter checklist