-
Notifications
You must be signed in to change notification settings - Fork 16
chore: add codebuild release #232
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
Conversation
df705c0
to
07c20df
Compare
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.
Changes originally in #234 lgtm
|
||
## 3.0.0-preview-1 2023-06-08 | ||
|
||
### Features |
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.
### Features | |
### Developer Preview |
- rm -rf ~/.m2/repository/software/amazon/cryptography/aws-cryptographic-material-providers | ||
- make transpile_implementation_java | ||
- make transpile_test_java | ||
- make mvn_local_deploy |
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.
just confirming that mvn_local_deploy will end up running the Java tests?
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.
mvn_local_deploy
won't run the Java tests; which is why we run them on make test_java and then run the long tests on line 60.
Now that you mention it they are out of order but if at any point they fail we won't publish.
Issue #, if available:
Description of changes:
depends on #234
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.