Skip to content

Preparation for v2024.2.1 release (CLI)#426

Merged
matsduf merged 3 commits intozonemaster:release/v2024.2.1from
matsduf:preparation-release-v2024.2.1
Mar 4, 2025
Merged

Preparation for v2024.2.1 release (CLI)#426
matsduf merged 3 commits intozonemaster:release/v2024.2.1from
matsduf:preparation-release-v2024.2.1

Conversation

@matsduf
Copy link
Contributor

@matsduf matsduf commented Mar 3, 2025

Purpose

This PR is a preparation for v2024.2.1 release.

Changes

  • Changes
  • lib/Zonemaster/CLI.pm
  • Makefile.PL

How to test this PR

Review.

@matsduf matsduf added the P-High Priority: Issue to be solved before other label Mar 3, 2025
@matsduf matsduf added this to the v2024.2.1 milestone Mar 3, 2025
Co-authored-by: Mattias Päivärinta <mattias@paivarinta.se>
tgreenx
tgreenx previously approved these changes Mar 3, 2025
Copy link
Contributor

@tgreenx tgreenx left a comment

Choose a reason for hiding this comment

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

What about #416? I see no mention of that here.

@tgreenx tgreenx dismissed their stale review March 3, 2025 19:35

Comment

@matsduf
Copy link
Contributor Author

matsduf commented Mar 3, 2025

What about #416? I see no mention of that here.

No, and that PR is not listed in https://github.com/zonemaster/zonemaster-cli/commits/release/v2024.2.1, which I used to find all PRs in the release. But the single commit c3b90b7 from the PR is in the listing. I guess that @marc-vanderwal merged that without merge commit.

@matsduf matsduf requested review from mattias-p and tgreenx March 3, 2025 20:52
@matsduf
Copy link
Contributor Author

matsduf commented Mar 3, 2025

What about #416? I see no mention of that here.

Fixed.

@matsduf matsduf merged commit 0271d5a into zonemaster:release/v2024.2.1 Mar 4, 2025
1 check passed
@matsduf matsduf deleted the preparation-release-v2024.2.1 branch March 4, 2025 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P-High Priority: Issue to be solved before other

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants