Skip to content

feat(release): fixed the release, added ci.yml#61

Merged
uibm merged 6 commits into
masterfrom
fix-release
Jan 15, 2025
Merged

feat(release): fixed the release, added ci.yml#61
uibm merged 6 commits into
masterfrom
fix-release

Conversation

@uibm
Copy link
Copy Markdown
Member

@uibm uibm commented Jan 7, 2025

No description provided.

uibm added 2 commits January 7, 2025 00:00
Signed-off-by: Ujjwal Kumar <Ujjwal.Kumar1@ibm.com>
Signed-off-by: Ujjwal Kumar <Ujjwal.Kumar1@ibm.com>
@astha-jain
Copy link
Copy Markdown
Member

@ujjwal-ibm We have a bunch of default travis environment variables that needs taken care of along with build scripts. Needs migration to actions.

@astha-jain
Copy link
Copy Markdown
Member

@astha-jain
Copy link
Copy Markdown
Member

Signed-off-by: Ujjwal Kumar <Ujjwal.Kumar1@ibm.com>
@uibm uibm marked this pull request as ready for review January 10, 2025 15:03
Copy link
Copy Markdown
Member

@astha-jain astha-jain left a comment

Choose a reason for hiding this comment

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

Signed-off-by: Ujjwal Kumar <Ujjwal.Kumar1@ibm.com>
Copy link
Copy Markdown
Member

@astha-jain astha-jain left a comment

Choose a reason for hiding this comment

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

I think we'll need to encrypt the keys instead of directly importing private key. This aligns with our previous process as well.

Comment thread .github/workflows/ci.yml
GH_TOKEN: ${{ secrets.GH_TOKEN }}
OSSRH_USERNAME: ${{ secrets.OSSRH_USERNAME }}
OSSRH_PASSWORD: ${{ secrets.OSSRH_PASSWORD }}
GPG_PRIVATE_KEY: ${{ secrets.GPG_PRIVATE_KEY }}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Signed-off-by: Ujjwal Kumar <Ujjwal.Kumar1@ibm.com>
Signed-off-by: Ujjwal Kumar <Ujjwal.Kumar1@ibm.com>
Copy link
Copy Markdown
Member

@astha-jain astha-jain left a comment

Choose a reason for hiding this comment

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

Looks good for now and not a blocker for publishing the SDK. I also want to continue researching on encrypting the private key and also want to look for SDK Core's guidance on it.

@uibm uibm merged commit 000ebb2 into master Jan 15, 2025
github-actions Bot pushed a commit that referenced this pull request Jan 15, 2025
# [0.23.0](0.22.0...0.23.0) (2025-01-15)

### Features

* **release:** fixed ci.yml ([#62](#62)) ([de68af4](de68af4))
* **release:** fixed the release, added ci.yml ([#61](#61)) ([000ebb2](000ebb2))
* **spec:** updated the sdk as per the api spec released on 2024-12-26 ([#60](#60)) ([77c9d37](77c9d37))
@ibm-vpc
Copy link
Copy Markdown
Collaborator

ibm-vpc commented Jan 15, 2025

🎉 This PR is included in version 0.23.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

github-actions Bot pushed a commit that referenced this pull request Jan 16, 2025
# [0.23.0](0.22.0...0.23.0) (2025-01-16)

### Features

* cleaned up unwanted changes ([#65](#65)) ([d0aa7ed](d0aa7ed))
* **release:** fixed ci.yml ([#62](#62)) ([de68af4](de68af4))
* **release:** fixed release maven changes ([#63](#63)) ([0a3d31e](0a3d31e))
* **release:** fixed the release, added ci.yml ([#61](#61)) ([000ebb2](000ebb2))
* **spec:** updated the sdk as per the api spec released on 2024-12-26 ([#60](#60)) ([77c9d37](77c9d37))
@uibm uibm deleted the fix-release branch January 16, 2025 09:42
github-actions Bot pushed a commit that referenced this pull request Jan 16, 2025
# [0.23.0](0.22.0...0.23.0) (2025-01-16)

### Features

* cleaned up unwanted changes ([#65](#65)) ([d0aa7ed](d0aa7ed))
* **release:** fixed ci.yml ([#62](#62)) ([de68af4](de68af4))
* **release:** fixed mvn versions issue ([#66](#66)) ([e71a131](e71a131))
* **release:** fixed release maven changes ([#63](#63)) ([0a3d31e](0a3d31e))
* **release:** fixed the release, added ci.yml ([#61](#61)) ([000ebb2](000ebb2))
* **spec:** updated the sdk as per the api spec released on 2024-12-26 ([#60](#60)) ([77c9d37](77c9d37))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants