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

Update UPDATING in main #712

Closed
wants to merge 5 commits into from
Closed

Conversation

grahamperrin
Copy link
Contributor

@grahamperrin grahamperrin commented Apr 12, 2023

Uppercase H for the FreeBSD Handbook.

Change lowercase current to uppercase CURRENT.

Remove the white space that precedes the link to the relevant section of the Handbook. This is subtle; it improves usability for people who habitually triple-click to select a chunk of text (compared to a double-click to select a word).

Uppercase H for the FreeBSD Handbook.

Remove the white space that precedes a URI.
@grahamperrin grahamperrin marked this pull request as ready for review April 12, 2023 02:00
@emaste
Copy link
Member

emaste commented May 1, 2023

IMO The whitespace before the URL is intentional, to make it stand out.

UPDATING Outdated Show resolved Hide resolved
UPDATING Outdated Show resolved Hide resolved
@bsdimp
Copy link
Member

bsdimp commented May 1, 2023

Only the capital h looks good to me.

@grahamperrin
Copy link
Contributor Author

IMO The whitespace before the URL is intentional, to make it stand out.

I think it's fairly outstanding as a separate paragraph for the address alone, but no argument.

image

@bsdimp
Copy link
Member

bsdimp commented May 4, 2023

NAK means "I'm explicitly saying no." so no to adding the version of FreeBSD to the -current UPDATING page...

The extra spacing would be fine if there's blank lines... the URL I think is now longer than it was when it was added (or I'm just confused).

FreeBSD-CURRENT, consistent with usage at pages such as <https://www.freebsd.org/releases/>. 

<freebsd#712 (comment)> relates.
grahamperrin added a commit to grahamperrin/freebsd-src that referenced this pull request May 8, 2023
grahamperrin added a commit to grahamperrin/freebsd-src that referenced this pull request May 8, 2023
@grahamperrin
Copy link
Contributor Author

… the URL I think is now longer than it was when it was added …

I don't know the back history, but URLs that included en_US.ISO8859-1 were longer. See for example:

https://github.com/freebsd/freebsd-src/pull/713/files#diff-7a99811ebf6b6268a2581fcf050df0a2c325fdbf1c02481dad16afd38f4bd6a1R9

freebsd-git pushed a commit that referenced this pull request May 14, 2023
Reviewed by: imp
Pull Request: #712
@grahamperrin grahamperrin deleted the patch-1 branch May 15, 2023 00:28
@emaste emaste added the merged label Jun 12, 2023
bsdjhb pushed a commit to bsdjhb/cheribsd that referenced this pull request Aug 1, 2023
Reviewed by: imp
Pull Request: freebsd/freebsd-src#712
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants