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

Update run documentation #683

Merged
merged 2 commits into from
Jul 10, 2019
Merged

Conversation

cchantep
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Jul 10, 2019

Codecov Report

Merging #683 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #683   +/-   ##
=======================================
  Coverage   61.03%   61.03%           
=======================================
  Files          78       78           
  Lines        1065     1065           
  Branches       22       22           
=======================================
  Hits          650      650           
  Misses        415      415

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3e253ca...4162d99. Read the comment docs.

docs/running.md Outdated

```bash
--env-var BINTRAY_USER=username \
--env-var BINTRAY_PASS=password
```

These variables will be accessible (in sbt) to all of the projects that Scala Steward checks dependencies for.
These variables will be accessible (in SBT) to all of the projects that Scala Steward checks dependencies for.
Copy link
Member

Choose a reason for hiding this comment

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

It should just be sbt. :-)

Suggested change
These variables will be accessible (in SBT) to all of the projects that Scala Steward checks dependencies for.
These variables will be accessible (in sbt) to all of the projects that Scala Steward checks dependencies for.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Weird naming ... updated

@fthomas
Copy link
Member

fthomas commented Jul 10, 2019

Many thanks @cchantep! I think we can close #48 with this update.

@cchantep
Copy link
Contributor Author

Ok, thx. Just had the root question.

@fthomas fthomas added this to the 0.3.0 milestone Jul 10, 2019
@fthomas fthomas merged commit ef49e22 into scala-steward-org:master Jul 10, 2019
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.

2 participants