Skip to content

Add section numbers that were missing in CMS_add0_cert.pod#20398

Closed
paulidale wants to merge 1 commit intoopenssl:openssl-3.1from
paulidale:doc-link-fix
Closed

Add section numbers that were missing in CMS_add0_cert.pod#20398
paulidale wants to merge 1 commit intoopenssl:openssl-3.1from
paulidale:doc-link-fix

Conversation

@paulidale
Copy link
Contributor

Some section numbers were missing which annoyed doc-nits.
This addresses this by putting them in.

  • documentation is added or updated
  • tests are added or updated

@paulidale paulidale added approval: review pending This pull request needs review by a committer approval: otc review pending triaged: documentation The issue/pr deals with documentation (errors) branch: 3.0 Applies to openssl-3.0 branch branch: 3.1 Applies to openssl-3.1 (EOL) labels Feb 28, 2023
@paulidale paulidale self-assigned this Feb 28, 2023
@t8m
Copy link
Member

t8m commented Feb 28, 2023

@paulidale this was just not properly cherry picked to 3.1 and 3.0 in #20369

@paulidale
Copy link
Contributor Author

The doc-nits change would be considered a feature rather than a bug fix IMO. We shouldn't merge that.

@DDvO DDvO added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Feb 28, 2023
@DDvO DDvO changed the title Add sections that were missing Add section numbers that were missing in CMS_add0_cert.pod Feb 28, 2023
@paulidale
Copy link
Contributor Author

Merged, thanks for the reviews.

@paulidale paulidale closed this Mar 1, 2023
@paulidale paulidale deleted the doc-link-fix branch March 1, 2023 08:55
openssl-machine pushed a commit that referenced this pull request Mar 1, 2023
Reviewed-by: David von Oheimb <[email protected]>
Reviewed-by: Matt Caswell <[email protected]>
(Merged from #20398)

(cherry picked from commit 2bb9e44)
openssl-machine pushed a commit that referenced this pull request Mar 1, 2023
Reviewed-by: David von Oheimb <[email protected]>
Reviewed-by: Matt Caswell <[email protected]>
(Merged from #20398)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: done This pull request has the required number of approvals branch: 3.0 Applies to openssl-3.0 branch branch: 3.1 Applies to openssl-3.1 (EOL) triaged: documentation The issue/pr deals with documentation (errors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants