Skip to content

Handle # api change in aws-sdk-go 1.44.46 #140

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jul 11, 2022

Conversation

AustinSiu
Copy link
Contributor

Issue #, if available: #137

Description of changes:

  • Updated aws-sdk-go version to latest.
  • Updated # document parsing logic to accommodate api changes introduced in v1.44.46 of aws-sdk-go.

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

@AustinSiu AustinSiu requested a review from a team as a code owner July 11, 2022 19:26
Copy link
Contributor

@bwagner5 bwagner5 left a comment

Choose a reason for hiding this comment

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

lgtm

@AustinSiu AustinSiu merged commit cec208c into aws:main Jul 11, 2022
hakman added a commit to hakman/amazon-ec2-instance-selector that referenced this pull request Jul 21, 2022
bwagner5 pushed a commit that referenced this pull request Jul 21, 2022
* Revert "Handle # api change in aws-sdk-go 1.44.46 (#140)"

This reverts commit cec208c.

* Update aws-sdk-go to v1.44.59
digocorbellini pushed a commit to digocorbellini/amazon-ec2-instance-selector that referenced this pull request Jul 21, 2022
* Revert "Handle # api change in aws-sdk-go 1.44.46 (aws#140)"

This reverts commit cec208c.

* Update aws-sdk-go to v1.44.59
digocorbellini added a commit that referenced this pull request Jul 26, 2022
* created sorting system based on json pathing

* incorporated sorting system into CLI

* added shorthand sorting fields

* Revert "Handle # api change in aws-sdk-go 1.44.46 (#145)

* Revert "Handle # api change in aws-sdk-go 1.44.46 (#140)"

This reverts commit cec208c.

* Update aws-sdk-go to v1.44.59

* added sorting example and new help printout to readme

* added jsonpath library license to third party licenses

* changed sorting to use sort.Slice()

* removed need for $ in json path

* shortened sort-by flag description and updated readme

* early sort return for empty and 1 element lists

* added sorter tests

* fixed typos

* reduced scope of if statement in main

Co-authored-by: Brandon Wagner <bmwagner10@gmail.com>

* simplified call to sort instances

* modified tests for new sorting interface

* fixed comment typo

* refactored sorting keys map

Co-authored-by: Rodrigo Okamoto <rodocp@amazon.com>
Co-authored-by: Ciprian Hacman <ciprian@hakman.dev>
Co-authored-by: Brandon Wagner <bmwagner10@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants