Skip to content

Removes mention of engines that are no longer supported as per 4.0 release.#117

Open
fwh-dc wants to merge 3 commits into
openssl:masterfrom
fwh-dc:rm-engine
Open

Removes mention of engines that are no longer supported as per 4.0 release.#117
fwh-dc wants to merge 3 commits into
openssl:masterfrom
fwh-dc:rm-engine

Conversation

@fwh-dc
Copy link
Copy Markdown

@fwh-dc fwh-dc commented Apr 18, 2026

No description provided.

Copy link
Copy Markdown
Member

@beldmit beldmit left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread policies/testing.md Outdated
@t8m t8m requested review from levitte, mattcaswell and t-j-h April 20, 2026 11:46
Copy link
Copy Markdown
Member

@beldmit beldmit left a comment

Choose a reason for hiding this comment

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

LGTM

@levitte
Copy link
Copy Markdown
Member

levitte commented Apr 21, 2026

The document starts with this:

This applies to all [stable] and development branches of the main code repository.

As long as we have public pre-4.0 release branches, where we do run tests, it is wrong to remove engines from this policy document.

Strong 👎🏼

Copy link
Copy Markdown
Member

@levitte levitte left a comment

Choose a reason for hiding this comment

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

I disapprove

@fwh-dc
Copy link
Copy Markdown
Author

fwh-dc commented Apr 21, 2026

That's a good point. This should wait to be merged until 8th of april 2030.

@beldmit
Copy link
Copy Markdown
Member

beldmit commented Apr 21, 2026

Should we instead update the wording?

Comment thread policies/testing.md
Comment thread policies/testing.md Outdated
Comment thread policies/testing.md Outdated
@fwh-dc
Copy link
Copy Markdown
Author

fwh-dc commented Apr 21, 2026

Fixed wording now.

@t8m t8m requested a review from levitte April 22, 2026 12:18
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.

5 participants