Skip to content
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

CI: Build, test, deploy to maven #11

Merged
merged 6 commits into from
Apr 27, 2023
Merged

CI: Build, test, deploy to maven #11

merged 6 commits into from
Apr 27, 2023

Conversation

jaeseung-bae
Copy link
Contributor

@jaeseung-bae jaeseung-bae commented Apr 26, 2023

Description

  • Close Add CI #3
  • Add CI/CD
    • Build, test, deploy to maven

Motivation and context

How has this been tested?

Screenshots (if appropriate):

Checklist:

  • I have added a relevant changelog to CHANGELOG.md.
  • I have added tests to cover my changes.
  • I have updated the documentation accordingly.

@jaeseung-bae jaeseung-bae marked this pull request as ready for review April 26, 2023 02:55
@jaeseung-bae jaeseung-bae self-assigned this Apr 26, 2023
Copy link
Member

@zemyblue zemyblue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change the title.
And please add prefix ci into the title in this case.

crypto/build.gradle.kts Outdated Show resolved Hide resolved
@jaeseung-bae jaeseung-bae changed the title Feat/ci CI: Build, test, deploy to maven Apr 26, 2023
@@ -37,7 +37,7 @@ dependencies {
testRuntimeOnly("org.junit.jupiter:junit-jupiter-engine:$jupiterVersion")
}

java{
java {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this change does not applied in tx/build.gradle.kts.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I moved maven-related definition into publishing blocks in both tx/ and crypto/.
Do you mean this??

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean line 18 in tx/buid.gradle.kts, there is no blank between java{.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok~

loin3
loin3 previously approved these changes Apr 27, 2023
.github/workflows/tag.yml Outdated Show resolved Hide resolved
zemyblue
zemyblue previously approved these changes Apr 27, 2023
@jaeseung-bae jaeseung-bae merged commit 512cc7e into main Apr 27, 2023
4 checks passed
@jaeseung-bae jaeseung-bae deleted the feat/ci branch April 27, 2023 06:29
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.

Add CI
3 participants