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

Recommend intellij plugin #26

Merged
merged 10 commits into from
Oct 16, 2019
Merged

Conversation

ferozco
Copy link
Contributor

@ferozco ferozco commented Oct 16, 2019

Before this PR

After this PR

==COMMIT_MSG==
When using the gradle plugin, users will receive a notification to install the IntelliJ plugin
==COMMIT_MSG==

Possible downsides?

@changelog-app
Copy link

changelog-app bot commented Oct 16, 2019

Generate changelog in changelog/@unreleased

Type

  • Feature
  • Improvement
  • Fix
  • Break
  • Deprecation
  • Manual task
  • Migration

Description

When using the gradle plugin, users will receive a notification to install the IntelliJ plugin

Check the box to generate changelog(s)

  • Generate changelog entry

module {
sourceDirs += sourceSets.main.groovy.srcDirs
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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


public class ConfigureExternalDependenciesXml extends UpdateIntellijXmlTask {
public ConfigureExternalDependenciesXml() {
getOutputFile().set(getProject().file(".idea/externalDependencies.xml"));
Copy link
Contributor

Choose a reason for hiding this comment

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

rename getOutputFile -> intellijXmlFile or something? emphasise it's not a pure gradle concept

@dansanduleac dansanduleac merged commit e97832a into develop Oct 16, 2019
@dansanduleac dansanduleac deleted the fo/recommend-intellij-plugin branch October 16, 2019 18:31
@svc-autorelease
Copy link
Collaborator

Released 0.2.2

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

Successfully merging this pull request may close these issues.

None yet

4 participants