Skip to content

Conversation

@sbernauer
Copy link
Member

@sbernauer sbernauer commented May 16, 2022

Description

As discussed today

Review Checklist

  • Code contains useful comments
  • (Integration-)Test cases added (or not applicable)
  • Documentation added (or not applicable)
  • Changelog updated (or not applicable)
  • Cargo.toml only contains references to git tags (not specific commits or branches)

Once the review is done, comment bors r+ (or bors merge) to merge. Further information

Copy link
Member

@maltesander maltesander left a comment

Choose a reason for hiding this comment

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

LGTM!

@sbernauer
Copy link
Member Author

If it's okay for you i would merge it tomorrow to let people have the time to have an objection?

@sbernauer
Copy link
Member Author

bors r+

bors bot pushed a commit that referenced this pull request May 17, 2022
…olume struct (#405) ## Description As discussed today
@bors
Copy link
Contributor

bors bot commented May 17, 2022

Build failed:

@sbernauer
Copy link
Member Author

bors retry

bors bot pushed a commit that referenced this pull request May 17, 2022
…olume struct (#405) ## Description As discussed today
@bors
Copy link
Contributor

bors bot commented May 17, 2022

Build failed:

@sbernauer
Copy link
Member Author

bors r+

@sbernauer
Copy link
Member Author

bors cancel

@bors
Copy link
Contributor

bors bot commented May 17, 2022

Canceled.

@sbernauer
Copy link
Member Author

bors r+

bors bot pushed a commit that referenced this pull request May 17, 2022
…olume struct (#405) ## Description As discussed today
@bors
Copy link
Contributor

bors bot commented May 17, 2022

Pull request successfully merged into main.

Build succeeded:

@bors bors bot changed the title Change credentials of S3ConnectionSpec to use the common SecretClassVolume struct [Merged by Bors] - Change credentials of S3ConnectionSpec to use the common SecretClassVolume struct May 17, 2022
@bors bors bot closed this May 17, 2022
@bors bors bot deleted the s3-credentials branch May 17, 2022 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants