Skip to content

Cut release 1.1.2#109

Merged
nywilken merged 2 commits intomainfrom
release/1.1.2
Jan 17, 2024
Merged

Cut release 1.1.2#109
nywilken merged 2 commits intomainfrom
release/1.1.2

Conversation

@nywilken
Copy link
Contributor

  • version: Cut release 1.1.2
  • version: prepare for 1.1.3 dev

@nywilken nywilken requested a review from a team as a code owner January 17, 2024 18:50
Copy link
Contributor

@lbajolet-hashicorp lbajolet-hashicorp left a comment

Choose a reason for hiding this comment

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

LGTM!

// (empty string) then it means that it is a final release. Otherwise, this
// is a pre-release such as "dev" (in development), "beta", "rc1", etc.
VersionPrerelease = "dev"
VersionPrerelease = ""
Copy link
Contributor

Choose a reason for hiding this comment

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

I would suggest updating the README for the plugin so the exerpt for required_plugins shows the latest version as 1.1.2, but this is optional.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah good point. I will push that and merge. Thanks. I'll do this for the other two plugins as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Actually it looks like they may be out of sync a bit. I will fast follow with a different PR since the integration install docs use ~> 1 for the version string.

@nywilken nywilken merged commit 6e37c6f into main Jan 17, 2024
@nywilken nywilken deleted the release/1.1.2 branch January 17, 2024 19:06
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