Skip to content

Conversation

@matsduf
Copy link
Contributor

@matsduf matsduf commented Jun 25, 2025

Purpose

This PR is a preparation for v2025.1 release.

Changes

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

Note

CI will fail until zonemaster/zonemaster-ldns#230

How to test this PR

Review.

@matsduf matsduf added this to the v2025.1 milestone Jun 25, 2025
@matsduf matsduf added the P-High Priority: Issue to be solved before other label Jun 25, 2025
Changes Outdated
v8.0.0 2025-06-26 (part of Zonemaster v2025.1 release)

[Breaking change]
- Improves performance by optimizing critical code sections #1420
Copy link
Member

Choose a reason for hiding this comment

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

While the performance improvement is the motivation for the PR, the breaking change is the change of the IPv6 format. Maybe list #1420 in both the "Breaking changes" and "Features" sections with different descriptions?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will fix.

Changes Outdated

v8.0.0 2025-06-26 (part of Zonemaster v2025.1 release)

[Breaking change]
Copy link
Member

Choose a reason for hiding this comment

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

The other categories are in plural.

Suggested change
[Breaking change]
[Breaking changes]

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The other categories are in plural.

For some reason this has always bin in singular. I'll update, but that will be a breaking change. :-)

Changes Outdated
- Updates translations (Danish, Norwegian) #1418, #1449
- Updates Dockerfile for release 2025.1 #1460
- Fixes test case DNSSEC10 for name servers sharing the same IPs #1457
- Updates local copies of IANA special IP registries #1456
Copy link
Member

Choose a reason for hiding this comment

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

Isn't this a minor change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Maybe, maybe not. It will have no practical affect to change. I'll do that.

@matsduf matsduf force-pushed the preparation-release-v2025.1 branch from 8b93762 to 5c7b798 Compare June 25, 2025 13:56
@matsduf matsduf merged commit f41b340 into zonemaster:develop Jun 26, 2025
3 of 6 checks passed
@matsduf matsduf deleted the preparation-release-v2025.1 branch June 26, 2025 14:16
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.

3 participants