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

prepare changelog for v0.17.1 #192

Merged
merged 2 commits into from
Apr 18, 2024
Merged

prepare changelog for v0.17.1 #192

merged 2 commits into from
Apr 18, 2024

Conversation

vinay-gopalan
Copy link
Contributor

No description provided.

@vinay-gopalan vinay-gopalan requested review from a team and Zlaticanin April 17, 2024 21:15
@Zlaticanin
Copy link
Contributor

There is another PR open by @tomhjp. Should we combine the two?

@vinay-gopalan
Copy link
Contributor Author

Ah thanks for pointing that out, completely missed that! Going to close this and add a comment on #186 to include the updates from here

@vinay-gopalan
Copy link
Contributor Author

We have decided to move forward with the v0.17.1 release with just the panic fix in #191. This PR has been reopened for review

@vinay-gopalan vinay-gopalan merged commit e660c4f into main Apr 18, 2024
4 checks passed
@vinay-gopalan vinay-gopalan deleted the changelog/v0.17.1 branch April 18, 2024 23:05
Copy link
Contributor

@tomhjp tomhjp left a comment

Choose a reason for hiding this comment

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

LGTM, just one optional suggestion on phrasing

## v0.17.1

BUG FIXES:
* Add nil check for response when unassigning roles [[GH-191]](https://github.com/hashicorp/vault-plugin-secrets-azure/pull/191)
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like my comment got lost somehow. The suggestion I had meant to leave was to phrase this more directly in terms of impact users will see, e.g. "Avoid panic when unassigning roles", but it's very minor and not worth opening up another PR for 👍 I only wanted my review comment to make sense :-)

tomhjp added a commit that referenced this pull request Apr 30, 2024
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