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(StdLib): Go v0.0.1 release #1195

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

ShubhamChaturvedi7
Copy link
Contributor

@ShubhamChaturvedi7 ShubhamChaturvedi7 commented Jan 10, 2025

Issue #, if available:

Description of changes:

Initial release based on MPL 1.8.0.

Files to be reviewed:
README.md
LICENSE
CHANGELOG.md
externs (Copied over from checked-in code / Optional)
Makefile (Copied over from checked-in code / Optional)
go.mod (Copied over from checked-in code / Optional)

Other files are polymorphed files.
Squash/merge commit message, if applicable:

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

@ShubhamChaturvedi7 ShubhamChaturvedi7 requested a review from a team as a code owner January 10, 2025 21:54
@ShubhamChaturvedi7 ShubhamChaturvedi7 changed the title release(StdLib): v0.0.1 feat(StdLib): Go v0.0.1 release Jan 10, 2025
seebees
seebees previously approved these changes Jan 13, 2025
Copy link
Contributor

@seebees seebees left a comment

Choose a reason for hiding this comment

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

LGTM

@rishav-karanjit
Copy link
Member

rishav-karanjit commented Jan 13, 2025

Some overall things:

  1. Can we have a MCM to see how this fits in?
  2. How do we know if this code runs as it ran when it was inside runtime/go? Knowing this before releasing would help MCM technician/requester.
  3. What tag are we adding to do this release?

@ShubhamChaturvedi7
Copy link
Contributor Author

Some overall things:

1. Can we have a MCM to see how this fits in?

2. How do we know if this code runs as it ran when it was inside runtime/go? Knowing this before releasing would help MCM technician/requester?

3. What tag are we adding to do this release?

MCM is answered here: https://amzn-aws.slack.com/archives/G01813NHQBD/p1736791637977939?thread_ts=1736559063.433249&cid=G01813NHQBD

  1. MCM is planned later after this release. It'd be futile to have an MCM which doesn't cover all the steps and we are still discovering the steps.
  2. Ran go build
  3. Mentioned in the description v0.0.1

@rishav-karanjit
Copy link
Member

Mentioned in the description v0.0.1

Shouldn't it be prefixed with path from the runtime?
ref: https://go.dev/ref/mod#vcs-version

If a module is defined in a subdirectory within the repository, that is, the module subdirectory portion of the module path is not empty, then each tag name must be prefixed with the module subdirectory, followed by a slash.

Copy link
Member

@rishav-karanjit rishav-karanjit left a comment

Choose a reason for hiding this comment

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

Talked offline. The GitHub tag is prefixed with subdirectories.

Non blocking but it could be great if we could have a CI test to do go build to check if the copied code works correctly.

LGTM

@ShubhamChaturvedi7
Copy link
Contributor Author

Mentioned in the description v0.0.1

Shouldn't it be prefixed with path from the runtime? ref: https://go.dev/ref/mod#vcs-version

If a module is defined in a subdirectory within the repository, that is, the module subdirectory portion of the module path is not empty, then each tag name must be prefixed with the module subdirectory, followed by a slash.

Are you talking about git tag or Go tag? The git tag would be something like git tag smithy-dafny-standard-library/v0.0.1-alpha (in this case releases/go will be prefixed as well), and go version tag v0.0.1 so that users can do go get [email protected].

Here's an example to explain this: https://github.com/ShubhamChaturvedi7/aws-cryptographic-material-providers-library-go/releases/tag/smithy-dafny-standard-library%2Fv0.0.1-alpha

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