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

Documentation: retire Homebrew, point to coursier #1960

Merged
merged 6 commits into from
May 9, 2020

Conversation

kitbellew
Copy link
Collaborator

Also, make the coursier examples directly usable via copy-paste, update the installation link and use scala 2.13 in examples.

@kitbellew kitbellew requested review from olafurpg and poslegm May 7, 2020 22:48
docs/installation.md Outdated Show resolved Hide resolved
docs/installation.md Show resolved Hide resolved
@kitbellew kitbellew force-pushed the 1947 branch 2 times, most recently from 5f75ae8 to 8771ca0 Compare May 8, 2020 14:42
@kitbellew kitbellew requested a review from olafurpg May 8, 2020 14:52
brew upgrade scalafmt
```
The recommended way to install the scalafmt command line tool is with
[Coursier](#coursier), itself available via Homebrew.
Copy link
Member

Choose a reason for hiding this comment

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

I think it would be nice to have copy-paste friendly instructions for Homebrew users

The recommended way to install the scalafmt command line tool is with
[Coursier](#coursier), itself available via Homebrew.

```sh
brew install coursier/formulas/coursier
coursier install scalafmt
scalafmt --help
```

If necessary, make sure to follow the Coursier instructions for updating
`$PATH` so that the `scalafmt` binary becomes available in your terminal.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done.

kitbellew added 6 commits May 9, 2020 12:35
Also remove the curl install example, the coursier install page has all
the examples for each platform.
When copying the command for local execution, it's frequently useful to
change the location where scalafmt is written. Hence, specify it last.
The next commit will describe a better way.
@kitbellew kitbellew requested a review from olafurpg May 9, 2020 21:13
@kitbellew kitbellew merged commit b6eb1e6 into scalameta:master May 9, 2020
@kitbellew kitbellew deleted the 1947 branch May 9, 2020 22:04
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