-
Notifications
You must be signed in to change notification settings - Fork 6
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
Make a local release of the reference checker #189
Conversation
Until the JSpecify Reference Checker is released to Maven Central, on must first run: | ||
|
||
```` | ||
./gradlew PublishToMavenLocal |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would have guessed that this command would need to use ..
and the one on line 14 would need to use .
. I haven't tested that theory, but does that sound right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the number of dots is correct. I added more explanations to hopefully make it clearer.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, thank you, that completely clarified things for me.
…hecker into release-checker
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the review!
Until the JSpecify Reference Checker is released to Maven Central, on must first run: | ||
|
||
```` | ||
./gradlew PublishToMavenLocal |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the number of dots is correct. I added more explanations to hopefully make it clearer.
Co-authored-by: Chris Povirk <[email protected]>
Contains the changes from #188, merge that first.
This can already be used to run
./gradlew PublishToMavenLocal
and allows local usage.The new directory
usage-demo
contains a working example that depends on the local release.