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

feat(go): version file in the generated module directory #2492

Merged
merged 5 commits into from
Feb 2, 2021

Conversation

iliapolo
Copy link
Contributor

@iliapolo iliapolo commented Jan 28, 2021

Since golang version isn't available anywhere inside the module, we embed a version file so that consumers can access the version.

This is needed for publishing go modules using jsii-release without the user having to supply the version externally.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@iliapolo iliapolo marked this pull request as draft January 28, 2021 14:10
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Jan 28, 2021
@mergify
Copy link
Contributor

mergify bot commented Jan 28, 2021

The title of this Pull Request does not conform with [Conventional Commits] guidelines. It will need to be adjusted before the PR can be merged.
[Conventional Commits]: https://www.conventionalcommits.org

@iliapolo iliapolo changed the title feat(go): Embed a version file in the generated module feat(go): Version file in the generated module directory Jan 28, 2021
@iliapolo iliapolo marked this pull request as ready for review January 28, 2021 15:08
Copy link
Contributor

@eladb eladb left a comment

Choose a reason for hiding this comment

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

Looks solid

eladb pushed a commit to aws/constructs that referenced this pull request Jan 31, 2021
Use the latest version of jsii-release to publish go releases to https://github.com/aws/constructs-go (currently private).
Until aws/jsii#2492 is resolved, add a build step which creates a `version` file in the go publish directory.
eladb pushed a commit to aws/constructs that referenced this pull request Feb 1, 2021
Use the latest version of jsii-release to publish go releases to https://github.com/aws/constructs-go (currently private).
Until aws/jsii#2492 is resolved, add a build step which creates a `version` file in the go publish directory.
@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
  • Commit ID: a7ac092
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@RomainMuller RomainMuller changed the title feat(go): Version file in the generated module directory feat(go): version file in the generated module directory Feb 2, 2021
@mergify
Copy link
Contributor

mergify bot commented Feb 2, 2021

Thank you for contributing! ❤️ I will now look into making sure the PR is up-to-date, then proceed to try and merge it!

@mergify mergify bot added the pr/ready-to-merge This PR is ready to be merged. label Feb 2, 2021
@mergify mergify bot merged commit da3ea25 into main Feb 2, 2021
@mergify mergify bot deleted the epolon/golang-version-file branch February 2, 2021 16:29
@mergify
Copy link
Contributor

mergify bot commented Feb 2, 2021

Merging (with squash)...

@mergify mergify bot removed the pr/ready-to-merge This PR is ready to be merged. label Feb 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution/core This is a PR that came from AWS.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants