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: test vector help text #657

Merged
merged 1 commit into from
Sep 1, 2024
Merged

fix: test vector help text #657

merged 1 commit into from
Sep 1, 2024

Conversation

ajewellamz
Copy link
Contributor

Issue #, if available:

Description of changes:

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.

@ajewellamz ajewellamz requested a review from a team as a code owner September 1, 2024 14:17
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

@@ -241,9 +241,15 @@ module {:options "-functionSyntax:4"} GetOpt {
}
}

predicate IsHelp(args : Parsed)
Copy link
Contributor

Choose a reason for hiding this comment

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

I prefer Help?

@ajewellamz ajewellamz merged commit 0fedaf1 into main Sep 1, 2024
70 checks passed
@ajewellamz ajewellamz deleted the ajewell/fix-help branch September 1, 2024 16:10
josecorella pushed a commit that referenced this pull request Sep 10, 2024
* add ECDH error message for Rust ([#574](#574)) ([473a34a](473a34a))
* **DDB-Model:** DDB Supports 100 actions per Transaction ([#692](#692)) ([8a67843](8a67843))
* GetCurrentTimeStamp returns ISO8601 format ([#575](#575)) ([c07a51f](c07a51f))
* maintain order in test vectors for languages with parallel tests ([#641](#641)) ([8c8a38f](8c8a38f))
* Remove 4.4 DDB and KMS patches, abstract test to work on later Dafny versions ([#611](#611)) ([d51d648](d51d648))
* Remove uses of `:|` ([#618](#618)) ([f12fe5b](f12fe5b))
* test vector help text ([#657](#657)) ([0fedaf1](0fedaf1))

* bump dafny verification and code gen to dafny 4.8.0 ([#520](#520)) ([e16539e](e16539e))
* **post-release:** Change back to 1.5.1-SNAPSHOT ([09cd9a4](09cd9a4))
lucasmcdonald3 pushed a commit that referenced this pull request Sep 24, 2024
lucasmcdonald3 pushed a commit to lucasmcdonald3/aws-cryptographic-material-providers-library that referenced this pull request Sep 26, 2024
lucasmcdonald3 pushed a commit to lucasmcdonald3/aws-cryptographic-material-providers-library that referenced this pull request Oct 1, 2024
ajewellamz pushed a commit that referenced this pull request Dec 18, 2024
* feat: copy over raw smithy-go codegen (#655)

Co-authored-by: Shubham Chaturvedi <[email protected]>

* feat: remove not required codegen features (#657)

Co-authored-by: Shubham Chaturvedi <[email protected]>

* fix: prettier formatting (#659)

Co-authored-by: Shubham Chaturvedi <[email protected]>

* feat: modify smithy-go for smithy-dafny (#658)

* feat: modify smithy-go for smithy-dafny

* Update codegen/smithy-dafny-codegen/src/main/java/software/amazon/polymorph/smithygo/codegen/AddOperationShapes.java

Co-authored-by: Lucas McDonald <[email protected]>

---------

Co-authored-by: Shubham Chaturvedi <[email protected]>
Co-authored-by: Lucas McDonald <[email protected]>

* feat: Go nameresolvers (#667)

Co-authored-by: Shubham Chaturvedi <[email protected]>

* feat: Adds shape visitors for local service (#672)

* feat: Adds shape visitors for local service

* fix: PR Comments

* fix: PR Comments

---------

Co-authored-by: Shubham Chaturvedi <[email protected]>

* feat: Adds local service generators (#681)

Co-authored-by: Shubham Chaturvedi <[email protected]>

* feat: adds aws-sdk shapevisitors (#679)

* feat: Adds shape visitors for local service

* fix: PR Comments

* fix: PR Comments

* feat: adds aws-sdk shape visitors

* fix: Remove timestamp shape from localservice visitors

* fix: PR comments

---------

Co-authored-by: Shubham Chaturvedi <[email protected]>

* feat: adds aws-sdk service generators (#721)

* feat: Adds shape visitors for local service

* fix: PR Comments

* fix: PR Comments

* feat: adds aws-sdk shape visitors

* fix: Remove timestamp shape from localservice visitors

* fix: PR comments

* feat: adds aws-sdk service generators

---------

Co-authored-by: Shubham Chaturvedi <[email protected]>

* feat: Merge from Golang/dev (#732)

Co-authored-by: Shubham Chaturvedi <[email protected]>

* feat: smithy-go (#734)

* feat: Merge from Golang/dev

* feat: smithy-go

* Update codegen/smithy-dafny-codegen/build.gradle.kts

Co-authored-by: Tony Knapp <[email protected]>

* Update codegen/smithy-dafny-codegen/src/main/java/software/amazon/polymorph/smithygo/codegen/ValidationGenerator.java

Co-authored-by: Tony Knapp <[email protected]>

* Update codegen/smithy-dafny-codegen/src/main/java/software/amazon/polymorph/smithygo/codegen/ValidationGenerator.java

Co-authored-by: Tony Knapp <[email protected]>

* fix: PR comments

---------

Co-authored-by: Shubham Chaturvedi <[email protected]>
Co-authored-by: Tony Knapp <[email protected]>

* feat: smithy-go TestModels (#735)

* feat: smithy-go TestModels

* feat: smithy-go TestModels

* feat: smithy-go (#734)

* feat: Merge from Golang/dev

* feat: smithy-go

* Update codegen/smithy-dafny-codegen/build.gradle.kts

Co-authored-by: Tony Knapp <[email protected]>

* Update codegen/smithy-dafny-codegen/src/main/java/software/amazon/polymorph/smithygo/codegen/ValidationGenerator.java

Co-authored-by: Tony Knapp <[email protected]>

* Update codegen/smithy-dafny-codegen/src/main/java/software/amazon/polymorph/smithygo/codegen/ValidationGenerator.java

Co-authored-by: Tony Knapp <[email protected]>

* fix: PR comments

---------

Co-authored-by: Shubham Chaturvedi <[email protected]>
Co-authored-by: Tony Knapp <[email protected]>

* fix: PR comments

---------

Co-authored-by: Shubham Chaturvedi <[email protected]>
Co-authored-by: Tony Knapp <[email protected]>

* feat: Go Tests, CI, and Makefile (#736)

* feat: Go Tests, CI, and Makefile

* Update .github/workflows/test_models_go_tests.yml

Co-authored-by: Robin Salkeld <[email protected]>

* fix: PR comments

* fix: enable aws-sdk/*v2 tests

---------

Co-authored-by: Shubham Chaturvedi <[email protected]>
Co-authored-by: Robin Salkeld <[email protected]>

* fix: PR comments

---------

Co-authored-by: Shubham Chaturvedi <[email protected]>
Co-authored-by: Lucas McDonald <[email protected]>
Co-authored-by: Tony Knapp <[email protected]>
Co-authored-by: Robin Salkeld <[email protected]>
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.

2 participants