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

www/falkon: update package description #195

Closed
wants to merge 4 commits into from

Conversation

grahamperrin
Copy link
Contributor

It's no longer 'new-ish'.

Whilst here, rewrite things slightly.

It's no longer 'new-ish'.

Whilst here, rewrite things slightly.
@grahamperrin
Copy link
Contributor Author

@grahamperrin
Copy link
Contributor Author

@BSDKaffee
Copy link
Contributor

We generally avoid using double spaces after a full stop. The use of a single space is standard in modern writing. "Edition" is a noun. I would replace it with the verb "editing". The word "Manager" in "SSL Manager" should be lowercase since it used generically. Bulleted lists that don't complete the sentence stem should not be punctuated. I would also remove the hyphen before "qtonly" because that is the name of the flavor.

@grahamperrin

This comment was marked as resolved.

@grahamperrin
Copy link
Contributor Author

We generally avoid using double spaces after a full stop. The use of a single space is standard in modern writing.

Agreed, generally; with exceptions in some FreeBSD Project contexts – and the exceptions are not yet suitably documented. My freebsd/freebsd-src@edacf4b (removal of a double space) was treated as an error, and so on.

Briefly: can you think of any context where representation of a package description, making direct use of e.g. the pkg-descr file for a port, is not monospaced?

@BSDKaffee
Copy link
Contributor

Should the same be done for the CA certificates database? There's currently an uppercase C for Certificates.

Yes, it is a generic term. I'd make it singular too. "CA certificate database" sounds better.

Singular, and lowercase c, for certificate.

Resolve
freebsd#195 (comment)
@BSDKaffee
Copy link
Contributor

BSDKaffee commented Oct 18, 2023

Briefly: can you think of any context where representation of a package description, making direct use of e.g. the pkg-descr file for a port, is not monospaced?

Probably any port maintained by danfe@. He's a fellow grammar pedant, which I appreciate, but I disagree about double spaces after full stops. It's not wrong, just a dated writing style. I used to write that way myself, to be honest, but have kicked the habit.

@BSDKaffee
Copy link
Contributor

My freebsd/freebsd-src@edacf4b (removal of a double space) was treated as an error, and so on.

I don't know why removing a double space from UPDATING would cause an error. There is a great lack of consistency with regards to double space vs. monospace after a full stop in that file, even within some of the individual entries.

Revert to single spaces after full stops.

Resolves the double spacing aspect of
freebsd#195 (comment)

Noting
freebsd/freebsd-doc@6313bc8#r1299188132
and perceptible gaps in documentation such as the (FDP) FreeBSD
Documentation Project Primer for New Contributors.
@arrowd
Copy link
Contributor

arrowd commented Dec 26, 2023

Merged, thanks.

@arrowd arrowd closed this Dec 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants