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

bump vendored ECR plugin from v2.1.0 to 2.1.1 #741

Merged
merged 1 commit into from
Oct 19, 2020
Merged

bump vendored ECR plugin from v2.1.0 to 2.1.1 #741

merged 1 commit into from
Oct 19, 2020

Conversation

yob
Copy link
Contributor

@yob yob commented Oct 13, 2020

There's no significant changes in this bump (full diff here). The only change is support for an older version of bash that isn't included in the elastic stack AMIs.

Still, I thought it was worth updating ahead of the v5.0.0 release. At least next time we won't need to repeat the chore of checking if the v2.1.1 release has relevant changes for the elastic stack.

@yob yob requested a review from a team October 13, 2020 12:46
There's no significant changes in this bump (full diff at [1]). The only
change is support for an older version of bash that isn't included in
the elastic stack AMIs.

Still, I thought it was worth updating ahead of the v5.0.0 release. At
least next time we won't need to repeat the chore of checking if the
v2.1.1 release has relevant changes for the elastic stack.

[1] buildkite-plugins/ecr-buildkite-plugin@v2.1.0...v2.1.1
Copy link
Member

@JuanitoFatas JuanitoFatas left a comment

Choose a reason for hiding this comment

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

👍🏼

Copy link
Contributor

@chloeruka chloeruka left a comment

Choose a reason for hiding this comment

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

👍

@chloeruka chloeruka merged commit 645d0b4 into master Oct 19, 2020
@chloeruka chloeruka deleted the bump-ecr branch October 19, 2020 02:47
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