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

Release 2.0.0 #84

Merged
merged 1 commit into from
Feb 21, 2024
Merged

Conversation

h-haaks
Copy link
Contributor

@h-haaks h-haaks commented Feb 21, 2024

Pull Request (PR) description

Prepare 2.0.0 release

@h-haaks h-haaks requested a review from rwaffen February 21, 2024 08:09
Comment on lines +20 to +24
- Do not allow bootstrap tokens with a leading newline [\#80](https://github.com/voxpupuli/puppet-k8s/pull/80) ([olavst-spk](https://github.com/olavst-spk))
- Do not allow bootstrap tokens with a trailing newline [\#79](https://github.com/voxpupuli/puppet-k8s/pull/79) ([olavst-spk](https://github.com/olavst-spk))
Copy link
Member

Choose a reason for hiding this comment

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

I added the bugfix label, please regenerate the changelog

CHANGELOG.md Outdated
**Merged pull requests:**

- Use a template string for the crictl download URL [\#83](https://github.com/voxpupuli/puppet-k8s/pull/83) ([olavst-spk](https://github.com/olavst-spk))
- Make cni\_plugins download url configurable with a template string [\#82](https://github.com/voxpupuli/puppet-k8s/pull/82) ([olavst-spk](https://github.com/olavst-spk))
Copy link
Member

Choose a reason for hiding this comment

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

added the enhancement label


[Full Changelog](https://github.com/voxpupuli/puppet-k8s/compare/015d6134ae3d9b40ca539c9439a0de79374bc27a...v1.0.0)

**Breaking changes:**
Copy link
Member

Choose a reason for hiding this comment

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

looks odd that the whole section was missing 🤔


**Implemented enhancements:**

- Make cni\_plugins download url configurable with a template string [\#82](https://github.com/voxpupuli/puppet-k8s/pull/82) ([olavst-spk](https://github.com/olavst-spk))
Copy link
Member

Choose a reason for hiding this comment

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

Oh so this looks like a breaking change because it drops a parameter in a class that isn't private? In that case we have to do a 2.0.0 release.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I guess #83 is also a breaking change then?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added labels and changed metadata to 2.0.0

@h-haaks h-haaks changed the title Release 1.1.0 Release 2.0.0 Feb 21, 2024
@h-haaks h-haaks requested a review from bastelfreak February 21, 2024 08:57
@rwaffen
Copy link
Member

rwaffen commented Feb 21, 2024

i would give it a go now.... you ok @bastelfreak ?

Copy link
Member

@bastelfreak bastelfreak left a comment

Choose a reason for hiding this comment

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

looks good to me.

@h-haaks h-haaks merged commit 489ff71 into voxpupuli:master Feb 21, 2024
3 checks passed
@h-haaks h-haaks deleted the release-prep branch February 23, 2024 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants