Skip to content

Don't use "version" in "OpenSSL Version 1.1.0" etc#3938

Closed
richsalz wants to merge 1 commit intoopenssl:masterfrom
richsalz:rm-version-word
Closed

Don't use "version" in "OpenSSL Version 1.1.0" etc#3938
richsalz wants to merge 1 commit intoopenssl:masterfrom
richsalz:rm-version-word

Conversation

@richsalz
Copy link
Copy Markdown
Contributor

No description provided.

@richsalz richsalz added branch: master Applies to master branch approval: review pending This pull request needs review by a committer labels Jul 15, 2017
@richsalz richsalz self-assigned this Jul 15, 2017
@InfoHunter
Copy link
Copy Markdown
Member

Seems a little conflicting with your suggested here: #3895 (comment)

I guess I need to fix that to follow this PR : )

@richsalz
Copy link
Copy Markdown
Contributor Author

you are right, but we had a bit of discussion and 'majority rules' :)

Copy link
Copy Markdown
Contributor

@paulidale paulidale left a comment

Choose a reason for hiding this comment

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

Are the documentation style updates being documented somewhere (the wiki)?

@paulidale paulidale 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 Jul 16, 2017
levitte pushed a commit that referenced this pull request Jul 16, 2017
Reviewed-by: Paul Dale <paul.dale@oracle.com>
(Merged from #3938)
@richsalz
Copy link
Copy Markdown
Contributor Author

Thanks, merged. We don't have a manpage style guide (yet). I'm looking for one to use as a starting point. For now, the style rules are "must pass find-doc-nits" :)

@richsalz richsalz closed this Jul 16, 2017
@richsalz richsalz deleted the rm-version-word branch July 16, 2017 23:03
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: master Applies to master branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants