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

Adding org.ajoberstar.grgit Gradle plugin #187

Merged
merged 7 commits into from
Jul 30, 2017
Merged

Adding org.ajoberstar.grgit Gradle plugin #187

merged 7 commits into from
Jul 30, 2017

Conversation

ajoberstar
Copy link
Owner

This is being migrated from gradle-git. Differences from 1.7.x
gradle-git:

  • Allow applying to projects besides the rootProject
  • Warn if project was applied to root project and a subproject
  • Add a close task that finalizes all other tasks

This resolves #178.

This is being migrated from gradle-git. Differences from 1.7.x
gradle-git:

- Allow applying to projects besides the rootProject
- Warn if project was applied to root project and a subproject
- Add a close task that finalizes all other tasks

This resolves #178.
Just basic testing that the Git repo is found or not.
Using a task, requires finalizing all other tasks, which has the
potential to cause a cycle if there is some other plugin/logic trying to
do the same thing. A build listener meets the need by just delaying
until all tasks have executed.
Given that it doesn't replace the Closure syntax, without breaking
existing usage, this doesn't add much and just introduces more to
support and document.
}
prj.ext.grgit = grgit
}
} catch (Exception e) {
Copy link
Owner Author

Choose a reason for hiding this comment

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

MAJOR The type Exception should not be caught rule

@ajoberstar
Copy link
Owner Author

SonarQube analysis reported 1 issue

  • MAJOR 1 major

Watch the comments in this conversation to review them.

@ajoberstar ajoberstar merged commit 0d89bb0 into master Jul 30, 2017
@ajoberstar ajoberstar deleted the gradle branch July 30, 2017 14:36
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.

Migrate Gradle plugin into this project
1 participant