-
Notifications
You must be signed in to change notification settings - Fork 21
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
simplify release publication #50
simplify release publication #50
Conversation
@@ -57,10 +51,6 @@ jobs: | |||
- name: cargo test | |||
working-directory: ./CedarJavaFFI | |||
run: cargo test --verbose | |||
- name: Build CedarJava |
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.
Will build (and test) once the release action is working again.
@@ -34,18 +34,12 @@ jobs: | |||
contents: write | |||
strategy: | |||
matrix: | |||
os: [ubuntu-latest, macos-latest, windows-latest] |
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.
Will re-enable once the release action is working again
toolchain: | ||
- stable | ||
steps: | ||
- name: Checkout cedar-java | ||
uses: actions/checkout@v3 | ||
- name: Checkout cedar |
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.
We only release once published to crates.io, so we can get rid of this in all branches
Simplify the release publishing workflows.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.