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

Update README.md #88

Merged
merged 2 commits into from
Jun 5, 2021
Merged

Update README.md #88

merged 2 commits into from
Jun 5, 2021

Conversation

shestee
Copy link
Contributor

@shestee shestee commented May 31, 2021

Updated documentation on recently added "no version property" feature that allows deducing version based on annotated tag.

Added documentation on "no version" feature.
Copy link
Contributor

@mockitoguy mockitoguy left a comment

Choose a reason for hiding this comment

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

Nice! Thanks!

@@ -19,9 +19,9 @@ Encourage and help software developers set up their releases to be fully automat
# shipkit-auto-version Gradle Plugin
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you also update the "shipkit-auto-version Gradle Plugin" section? That section indicates that you need a version.properties file. Perhaps we can simplify this section to avoid duplication with "Usage" section.

README.md Outdated

## Usage
## USAGE
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: Let's use lowercase consistently (or uppercase), e.g. Usage here or CAPITAL everywhere else for headers

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for review! The documentation is now updated with your suggestions.

Simplified 'about' section of README.md and added short info about new feature. Details are contained in 'usage' section.
@mockitoguy mockitoguy merged commit e191dbe into master Jun 5, 2021
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.

3 participants