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

RFCs and references #71

Merged
merged 3 commits into from
Nov 23, 2022
Merged

RFCs and references #71

merged 3 commits into from
Nov 23, 2022

Conversation

nevans
Copy link
Collaborator

@nevans nevans commented Nov 21, 2022

For simple ag/ack/rg/grep during development, I've added even more RFCs to the rake file.

Additionally, I updated and reformatted the "References" section... again. See my comments on 981ea30.

@nevans nevans requested a review from shugo November 21, 2022 15:29
@nevans nevans added the documentation Improvements or additions to documentation label Nov 21, 2022
Switch to the official RFC citation format, as reported at each RFCs
txt citation URL, e.g: https://www.rfc-editor.org/refs/ref9051.txt.

Removed "EXT-" from any citations that were still using it, because that
is how modern IMAP RFCs cite their references.

Add links to relevant specifications for response data structs, without
resorting to the full citation syntax.

----

This is closer to what the format had already been before I switched it
up the last time (9cd562a).  I believe the earlier format was simply
copy/pasted from the references section of whichever RFC was being used
at the time (e.g. 2060 or 3501).  But the RFC citation format has
changed since then, too.

I'd previously changed it to be more "readable".  But since then I've
added links throughout the documentation of methods and data types, only
occasionally needing a list of "see also" links.  That approach works
and satisfies my readability concerns.  So our references list should
just be a standard list of citations and it would be better to use the
official citation format.

To consider: should we move the references list to a REFERENCES.md file?
@nevans
Copy link
Collaborator Author

nevans commented Nov 22, 2022

@shugo: the reference to nevans/rdoc has been removed from this PR.

You can see the rendered result here: https://nevans.github.io/net-imap/Net/IMAP.html#class-Net::IMAP-label-References

Copy link
Member

@shugo shugo left a comment

Choose a reason for hiding this comment

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

It looks fine!

@nevans nevans merged commit a3a88e4 into master Nov 23, 2022
@nevans nevans deleted the rfcs-and-references branch November 23, 2022 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Development

Successfully merging this pull request may close these issues.

2 participants