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

Generated Manifest of Build Artifacts #283

Merged
merged 5 commits into from
May 12, 2021

Conversation

skmcgrail
Copy link
Member

@skmcgrail skmcgrail commented Apr 6, 2021

Creates a generated manifest of the build output artifacts. Introduces the ability to handle selection of the minimum required standard library, and minimum version required for a given set of dependencies. This manifest will be used to ultimately remove the responsibilities of generating the go.mod from the smithy-go code base and move that concern into the SDK.

Example generated.json from the smithy-go-codegen-test test code:

{
    "dependencies": {
        "github.com/aws/smithy-go": "v1.3.0",
        "github.com/jmespath/go-jmespath": "v0.4.0"
    },
    "files": [
        "api_client.go",
        "api_op_BadName.go",
        "api_op_GetCity.go",
        "api_op_GetCityAnnouncements.go",
        "api_op_GetCityImage.go",
        "api_op_GetCurrentTime.go",
        "api_op_GetForecast.go",
        "api_op_ListCities.go",
        "doc.go",
        "generated.json",
        "go.mod",
        "types/enums.go",
        "types/errors.go",
        "types/types.go",
        "types/types_exported_test.go",
        "validators.go"
    ],
    "go": "1.15",
    "module": "weather"
}

@skmcgrail skmcgrail requested review from jasdel and skotambkar April 6, 2021 22:36
@skmcgrail skmcgrail force-pushed the manifestBuildArtifact branch from 3bbd461 to fc8d900 Compare May 5, 2021 00:56
@skmcgrail skmcgrail force-pushed the manifestBuildArtifact branch from fc8d900 to 44b6a8a Compare May 10, 2021 23:09
@skmcgrail skmcgrail merged commit bce8df7 into aws:main May 12, 2021
@skmcgrail skmcgrail deleted the manifestBuildArtifact branch November 8, 2021 18:05
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