Skip to content
This repository has been archived by the owner on Jan 18, 2021. It is now read-only.

More concise release notes #108

Merged
merged 6 commits into from May 4, 2017
Merged

More concise release notes #108

merged 6 commits into from May 4, 2017

Conversation

mockitoguy
Copy link
Member

Best to review commit-by-commit!

  1. Started using @wwilk's versionInfo.previousVersion, which helped to get rid of bunch of classes.
  2. Hooked up concise notes formatter which is way cleaner. However, it does not yet support @mstachniuk's stuff (contributor links, API calls avoidance). This is coming soon :)

@mockitoguy
Copy link
Member Author

Ping :)

@mockitoguy
Copy link
Member Author

mockitoguy commented May 4, 2017

Friendly ping 2 :) Try to take review commit-by-commit. It's up to you how much you want to review the impl details. I would recommend to focus on:

  • public API changes like new/changed properties on tasks
  • new plugins / wiring between Gradle plugins / dependsOn on tasks

I would recommend to defocus on:

  • impl details of leaf classes that actually do work
  • refactorings such as removing a lot of code, moving code around

Let me know if reviewing this PR is helpful for you to get more understanding about the project. If it isn't (and I will respect that), I will not push for reviews but wait a bit and merge myself :)

Thanks guys for working on this project together!!!

This new setting is great! Code will be so much cleaner.
New formatter offers much cleaner outlook. However, this breaks 2 features:
 - we no longer show 'publication repository'
 - we no longer show contributor links

I will address those issues in the next set of changes
Thank you team for adding some nice things (like previous version
in version.properties). This allowed us to get rid of lots of code :)
@mockitoguy
Copy link
Member Author

Thank you for review!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants