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

Make better rendering of general policy-supplementals #369

Closed

Conversation

levitte
Copy link
Member

@levitte levitte commented Oct 12, 2022

The quick method in the previous commit was incorrect. We change it to
align with how other policy files are rendered.

The quick method in the previous commit was incorrect.  We change it to
align with how other policy files are rendered.
@levitte levitte added the URGENT label Oct 12, 2022
@levitte levitte self-assigned this Oct 12, 2022
@levitte levitte requested review from a team October 12, 2022 08:45
@levitte
Copy link
Member Author

levitte commented Oct 12, 2022

The previous commit actually renders nothing.

I've tested that this PR does what it should.

@levitte
Copy link
Member Author

levitte commented Oct 12, 2022

Merged

061f8b1 Make better rendering of general policy-supplementals

@levitte levitte closed this Oct 12, 2022
@levitte levitte deleted the make-policy-supplemental-better branch October 12, 2022 10:40
openssl-machine pushed a commit that referenced this pull request Oct 12, 2022
The quick method in the previous commit was incorrect.  We change it to
align with how other policy files are rendered.

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from #369)
iamamoose pushed a commit that referenced this pull request Nov 7, 2022
The quick method in the previous commit was incorrect.  We change it to
align with how other policy files are rendered.

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from #369)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants