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

Add cljstyle 0.12.1 #80734

Merged
merged 2 commits into from
Apr 16, 2020
Merged

Add cljstyle 0.12.1 #80734

merged 2 commits into from
Apr 16, 2020

Conversation

robhanlon22
Copy link
Contributor

cljstyle is a tool for formatting Clojure code.

After making all changes to the cask:

  • brew cask audit --download cljstyle is error-free.
  • brew cask style --fix cljstyle reports no offenses.
  • The commit message includes the cask’s name and version.
  • The submission is for a stable version or documented exception.

Additionally, if adding a new cask:

Resolves greglook/cljstyle#36.

cljstyle is a tool for formatting Clojure code.
@vitorgalvao
Copy link
Member

Thank you, and welcome.

From faq/rejected_casks:

The app is both open-source and CLI-only (i.e. it only uses the binary artifact). In that case, and in the spirit of deduplication, submit it first to Homebrew/core as a formula that builds from source. If it is rejected, you may then try again as a cask (link us to the issue so we can see the discussion and reasoning for rejection).

@robhanlon22 robhanlon22 deleted the cljstyle branch April 16, 2020 22:18
@robhanlon22 robhanlon22 restored the cljstyle branch April 16, 2020 22:31
@robhanlon22
Copy link
Contributor Author

robhanlon22 commented Apr 16, 2020

Hi @vitorgalvao—I looked into building this from source, but it has a build-time dependency on GraalVM, which is cask-only: https://github.com/graalvm/homebrew-tap Any suggestions on where to go from here? Thanks.

@vitorgalvao vitorgalvao reopened this Apr 16, 2020
@vitorgalvao vitorgalvao merged commit b336c62 into Homebrew:master Apr 16, 2020
@robhanlon22
Copy link
Contributor Author

Thank you @vitorgalvao!!

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.

Setup via brew
2 participants