Skip to content
This repository has been archived by the owner on Apr 20, 2024. It is now read-only.

Release publishing flow #122

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Release publishing flow #122

wants to merge 9 commits into from

Conversation

vighneshiyer
Copy link
Collaborator

Drafting tag-based publishing flow for barstools. Example release here (https://repo1.maven.org/maven2/edu/berkeley/cs/barstools_2.12/).

@vighneshiyer vighneshiyer marked this pull request as ready for review August 5, 2022 23:13
Copy link
Contributor

@abejgonzalez abejgonzalez left a comment

Choose a reason for hiding this comment

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

Has this been tested by bumping the submodule in CY? I'm slightly concerned about the build.sbt settings changing (I think the settings being put into buildSettings and publishSettings caused problems in the past - though I forget).

Comment on lines +24 to +25
SONATYPE_PASSWORD: ${{ secrets.SONATYPE_PASSWORD }}
SONATYPE_USERNAME: ${{ secrets.SONATYPE_USERNAME }}
Copy link
Contributor

Choose a reason for hiding this comment

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

Who owns these? I'm concerned that these might be lost somewhere when one of the dev's leave.

- run: sudo apt update && sudo apt install -y gnupg
- run: echo $PGP_SECRET | base64 --decode | gpg --batch --import
env:
PGP_SECRET: ${{ secrets.PGP_SECRET }}
Copy link
Contributor

Choose a reason for hiding this comment

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

Ditto (see the next msg)

val defaultVersions = Map(
"chisel3" -> "3.5.1",
"chisel3" -> "3.5.3",
"chisel-iotesters" -> "2.5.1"
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: Should we just bump this to 2.5.4 anyways.

@@ -3,7 +3,7 @@ Barstools

---

![Test](https://github.com/freechipsproject/ucb-bar/barstools/Test/badge.svg)
![CI Status](https://github.com/ucb-bar/barstools/actions/workflows/publish.yml/badge.svg)
Copy link
Contributor

Choose a reason for hiding this comment

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

If we are publishing to Sonatype, we should also update the readme to mention what needs to be added to peoples build.sbt files.

@@ -1,3 +1,4 @@
addSbtPlugin("com.eed3si9n" % "sbt-assembly" % "0.14.5")
addSbtPlugin("org.scalameta" % "sbt-scalafmt" % "2.4.3")
addSbtPlugin("net.virtual-void" % "sbt-dependency-graph" % "0.9.2")
addSbtPlugin("io.shiftleft" % "sbt-ci-release-early" % "2.0.32")
Copy link
Contributor

Choose a reason for hiding this comment

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

Not related to this PR. I wonder if there is another plugin/CI-flow that auto-bumps the SBT versions, scala versions, etc (and if that is really needed).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants