Skip to content

feat: Added Gradle plugin#625

Open
chriptus13 wants to merge 3 commits intoohmybash:masterfrom
chriptus13:add-gradle-plugin
Open

feat: Added Gradle plugin#625
chriptus13 wants to merge 3 commits intoohmybash:masterfrom
chriptus13:add-gradle-plugin

Conversation

@chriptus13
Copy link
Copy Markdown

Added Gradle plugin similar to what we have in oh-my-zsh

Comment thread plugins/gradle/README.md
@@ -0,0 +1,25 @@
# Gradle plugin

This plugin adds completions and aliases for [Gradle](https://gradle.org/).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This plugin doesn't seem to provide completions. Is there a plan to add them?

Comment thread plugins/gradle/gradle.plugin.sh Outdated
Comment thread plugins/gradle/gradle.plugin.sh Outdated

function gradle-or-gradlew() {
# find project root
# taken from https://github.com/gradle/gradle-completion
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

That code seems to be licensed under the MIT license, meaning that the original copyright notice should be retained. Please include it.

@akinomyoga
Copy link
Copy Markdown
Contributor

similar to what we have in oh-my-zsh

I checked the upstream OMZ plugin and realized that this PR is actually not just similar but based on the OMZ plugin. In that case, could you also mention the upstream OMZ plugin in the code comment? Since the upstream is a moving target, it is better to keep the permalink that ours is based on:

https://github.com/ohmyzsh/ohmyzsh/blob/62cf1201b031399e7251abeee859e895ee825a48/plugins/gradle/gradle.plugin.zsh

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.

2 participants