Skip to content

Conversation

@matsduf
Copy link
Contributor

@matsduf matsduf commented Sep 1, 2023

Purpose

Preparation for release.

Changes

Updates files Changes and Engine.pm.

How to test this PR

Review the changes.

@matsduf matsduf added the P-High Priority: Issue to be solved before other label Sep 1, 2023
@matsduf matsduf added this to the v2023.1.4 milestone Sep 1, 2023
@matsduf
Copy link
Contributor Author

matsduf commented Sep 4, 2023

@tgreenx, is the description of #1282 in the updated Changes file correct? Also see #1282 (comment)

@marc-vanderwal
Copy link
Contributor

For the changelog, see my suggestion here: zonemaster/zonemaster#1201 (review)

@matsduf matsduf requested a review from mattias-p September 4, 2023 20:48
@matsduf
Copy link
Contributor Author

matsduf commented Sep 4, 2023

@mattias-p, @marc-vanderwal and @tgreenx, please (re-) review this PR, especially the updated Changes file.

Changes Outdated
v4.7.3 2023-08-07 (public fix version)

[Fixes]
- Fixes a bug (regression) in test case BASIC01 that made tests of zones
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove (regression)
Camel case test case

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@tgreenx, camel cased it would be e.g. tEst cAse.

@tgreenx
Copy link
Contributor

tgreenx commented Sep 5, 2023

For some reason I cannot make another review right now, but I just spotted that the date is wrong in the change log.

@matsduf
Copy link
Contributor Author

matsduf commented Sep 5, 2023

Date fixed.

@matsduf matsduf force-pushed the preparation-for-v2023.1.4 branch from fdef49f to a7bd003 Compare September 5, 2023 20:25
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.

Add #1285 here

@matsduf
Copy link
Contributor Author

matsduf commented Sep 7, 2023

@tgreenx, @mattias-p, @blacksponge, @PNAX, @marc-vanderwal -- please review.

@matsduf matsduf merged commit ab8b0a0 into zonemaster:releases/v2023.1.4 Sep 7, 2023
@matsduf matsduf deleted the preparation-for-v2023.1.4 branch September 7, 2023 09:46
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