Skip to content
This repository has been archived by the owner on Feb 11, 2022. It is now read-only.

Release v0.4 #29

Merged
merged 3 commits into from
Feb 22, 2017
Merged

Release v0.4 #29

merged 3 commits into from
Feb 22, 2017

Conversation

mr-archano
Copy link
Contributor

@mr-archano mr-archano commented Feb 22, 2017

Scope of the PR

We want to release a new version of the plugin on Novoda Bintray maven repo, to ship the changes from #27, #28 and #25.

Considerations and implementation

  • Bumped version for release
  • Updated README
  • Updated CHANGELOG

@mr-archano mr-archano changed the title Release/prepare v0.4 Release v0.4 Feb 22, 2017
README.md Outdated
@@ -128,5 +142,5 @@ Tool | Android | Java | Documentation |

#### Support for sharable configurations

We plan to add support for consuming rules (eg: configuration files for Checkstyle or PMD, default exclude filters, etc) via a
The plugin can consume rules (eg: configuration files for Checkstyle or PMD, default exclude filters, etc) via a
separate artifact you can share across projects. _More info to come._
Copy link
Contributor

Choose a reason for hiding this comment

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

Is "more info to come" still valid?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure. I'd like to expand this with an example, but we have no publicly deployed artifact for shared rules yet, hence the More info to come still there.

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe remove it until we have info?

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm happy to leave it then. It just reads as if the feature isn't implemented, which it is.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point guys. I'll get rid of it.

Copy link
Contributor

@ataulm ataulm left a comment

Choose a reason for hiding this comment

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

👍 also what Francesco said

@ouchadam
Copy link
Contributor

leaving to @frapontillo

@frapontillo frapontillo merged commit 16270db into master Feb 22, 2017
@frapontillo frapontillo deleted the release/prepare_v0.4 branch February 22, 2017 16:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants