Skip to content

Add source of the sbt-plugin #27

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

Merged
merged 4 commits into from
Jul 6, 2022
Merged

Add source of the sbt-plugin #27

merged 4 commits into from
Jul 6, 2022

Conversation

adpi2
Copy link
Member

@adpi2 adpi2 commented Jul 5, 2022

The sources are taken from https://github.com/scalacenter/sbt-github-dependency-graph, which should then be deprecated.

I think it is better to have the sbt-plugin and the Github action in the same repository because the sbt-plugin is an implementation detail of the Github action. It cannot be used outside the Github action.

The two are tightly coupled and must be released together.

@adpi2 adpi2 force-pushed the sbt-github-dependency-graph branch from bf03352 to 3b0b134 Compare July 5, 2022 15:34
@adpi2 adpi2 requested a review from sjrd July 5, 2022 15:40
Originally from https://github.com/scalacenter/sbt-github-dependency-graph
I think it is better to have the sbt-plugin and the Github action
in the same place because the sbt-plugin is a implementation detail
of the Github action. The sbt-plugin cannot be used outside the Github action.
The two are tightly coupled and must be released together.
@adpi2 adpi2 force-pushed the sbt-github-dependency-graph branch from 3b0b134 to 491ba08 Compare July 6, 2022 07:46
@adpi2 adpi2 force-pushed the sbt-github-dependency-graph branch from 3ffd040 to 37ce6bb Compare July 6, 2022 08:47
@adpi2 adpi2 force-pushed the sbt-github-dependency-graph branch from 9cfe9ce to 31e2e33 Compare July 6, 2022 09:01
@adpi2 adpi2 merged commit 2b77c74 into main Jul 6, 2022
@adpi2 adpi2 deleted the sbt-github-dependency-graph branch July 6, 2022 09:09
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