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

Setup Publishing & Dokka #13

Merged
merged 8 commits into from
Apr 30, 2023
Merged

Setup Publishing & Dokka #13

merged 8 commits into from
Apr 30, 2023

Conversation

nomisRev
Copy link
Contributor

@nomisRev nomisRev commented Apr 28, 2023

@franciscodr this sets up Gradle for publishing both SNAPSHOT and stable versions by manually triggering a Github Action.

All that is missing are the secrets on Github.

  OSS_USER: '${{ secrets.OSS_USER }}'
  OSS_TOKEN: '${{ secrets.OSS_TOKEN }}'
  OSS_STAGING_PROFILE_ID: '${{ secrets.OSS_STAGING_PROFILE_ID }}'
  SIGNING_KEY_NAME: '${{ secrets.SIGNING_KEY_NAME }}'
  SIGNING_KEY_ID: '${{ secrets.SIGNING_KEY_ID }}'
  SIGNING_KEY_PASSPHRASE: '${{ secrets.SIGNING_KEY_PASSPHRASE }}'
  SIGNING_KEY: '${{ secrets.SIGNING_KEY }}'

@nomisRev
Copy link
Contributor Author

@gutiory @realdavidvega ready for review

build.gradle.kts Outdated Show resolved Hide resolved
.github/workflows/snapshot.yml Outdated Show resolved Hide resolved
.github/workflows/snapshot.yml Outdated Show resolved Hide resolved
.github/workflows/snapshot.yml Outdated Show resolved Hide resolved
.github/workflows/snapshot.yml Outdated Show resolved Hide resolved
gradle.properties Outdated Show resolved Hide resolved
.github/workflows/publish.yml Outdated Show resolved Hide resolved
.github/workflows/publish.yml Outdated Show resolved Hide resolved
@nomisRev
Copy link
Contributor Author

@franciscodr I copied this from Arrow, can we do the same clean-up there?

@franciscodr
Copy link
Contributor

@franciscodr I copied this from Arrow, can we do the same clean-up there?

Yes, we should take a look into Arrow project to clean-up the builds

Copy link
Contributor

@franciscodr franciscodr left a comment

Choose a reason for hiding this comment

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

@nomisRev The required secrets are already set in the repository. I think we're ok to merge this pull request and publish our first snapshot. Thanks!

@nomisRev nomisRev merged commit ce16f04 into main Apr 30, 2023
@nomisRev nomisRev deleted the publishing branch April 30, 2023 19:16
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.

3 participants