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

fix: enable builds on windows #273

Merged
merged 2 commits into from
Aug 31, 2023
Merged

fix: enable builds on windows #273

merged 2 commits into from
Aug 31, 2023

Conversation

padamstx
Copy link
Member

PR summary

This PR updates the project to allow builds to be done on Windows.

PR Checklist

Please make sure that your PR fulfills the following requirements:

  • The commit message follows the Angular Commit Message Guidelines.
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@padamstx padamstx self-assigned this Aug 23, 2023
@padamstx padamstx force-pushed the add-windows-instructions branch 2 times, most recently from 563a6a7 to 960b63d Compare August 23, 2023 21:42
@padamstx padamstx marked this pull request as ready for review August 23, 2023 21:48
@padamstx
Copy link
Member Author

@michaelbeck Hi Michael, this is the PR for the Go SDK to provide a little more detail for Windows users. I don't think there was really all that much to change, other than including details of the prereqs (note that I am performing these steps in a powershell window within my Windows 11 VM to keep me honest 😂 ). Let me know if you think more details are needed. Thanks!

This commit updates the project to enable
builds to be performed on Windows.
I also bumped the go core version to the current latest.

Signed-off-by: Phil Adams <phil_adams@us.ibm.com>
Copy link
Member

@pyrooka pyrooka left a comment

Choose a reason for hiding this comment

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

Works well here! 👍

@padamstx padamstx merged commit b39d2d5 into main Aug 31, 2023
4 checks passed
@padamstx padamstx deleted the add-windows-instructions branch August 31, 2023 20:12
ibm-devx-sdk pushed a commit that referenced this pull request Aug 31, 2023
## [0.47.1](v0.47.0...v0.47.1) (2023-08-31)

### Bug Fixes

* enable builds on windows ([#273](#273)) ([b39d2d5](b39d2d5))
@ibm-devx-sdk
Copy link

🎉 This PR is included in version 0.47.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

manukm-ibm pushed a commit to manukm-ibm/platform-services-go-sdk that referenced this pull request Sep 13, 2023
This commit updates the project to enable
builds to be performed on Windows.
I also bumped the go core version to the current latest.

Signed-off-by: Phil Adams <phil_adams@us.ibm.com>
Signed-off-by: manu.k.m <manu.k.m@ibm.com>
manukm-ibm pushed a commit to manukm-ibm/platform-services-go-sdk that referenced this pull request Sep 13, 2023
## [0.47.1](IBM/platform-services-go-sdk@v0.47.0...v0.47.1) (2023-08-31)

### Bug Fixes

* enable builds on windows ([IBM#273](IBM#273)) ([b39d2d5](IBM@b39d2d5))

Signed-off-by: manu.k.m <manu.k.m@ibm.com>
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.

None yet

3 participants