-
Notifications
You must be signed in to change notification settings - Fork 119
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
build: use common Hiero Gradle config #2033
Conversation
3da628a
to
a2ee99a
Compare
a2ee99a
to
3590ef4
Compare
b638150
to
c8f142d
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.
Review applies to:
.github/workflows/build.yml
.github/workflows/release-artifacts.yml
292570a
to
0a47fed
Compare
0a47fed
to
5ae7eda
Compare
Signed-off-by: Jendrik Johannes <[email protected]>
5ae7eda
to
a749dbb
Compare
Hey @jjohannes , i will prioritise this PR and #2045 in the next milestone |
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.
Review applies to:
.github/dependabot.yml
.github/workflows/build.yml
.github/workflows/release-artifacts.yml
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.
LGTM! Thanks @jjohannes
@hashgraph/devops-ci @hashgraph/release-engineering-managers Would need a review on this one |
This now follows a similar setup as in hedera-services. This update was missed in #2033. Signed-off-by: Jendrik Johannes <[email protected]>
This now follows a similar setup as in hedera-services. This update was missed in #2033. Signed-off-by: Jendrik Johannes <[email protected]>
Description:
Before this can be integrated:
Related issue(s):
hashgraph/hedera-services#14255
#1794
#1669
#2054