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

Add codespell config, workflow to avoid adding typos. Fixed few typos still found #517

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

yarikoptic
Copy link

No description provided.

=== Do not change lines below ===
{
 "chain": [],
 "cmd": "codespell -w",
 "exit": 0,
 "extra_inputs": [],
 "inputs": [],
 "outputs": [],
 "pwd": "."
}
^^^ Do not change lines above ^^^
@ghost
Copy link

ghost commented Feb 21, 2024

Thank you for this pull request, however this document is now in the hands of the RFC Editor. I've checked the most recent edit of the document and the spelling errors you've found have already been fixed. Once the document has been published this repository will along with the rest of the organisation be archived and handed over as per RFC 8875.

@ghost ghost closed this Feb 21, 2024
@glyn
Copy link
Collaborator

glyn commented Feb 22, 2024

Thank you for this pull request, however this document is now in the hands of the RFC Editor. I've checked the most recent edit of the document and the spelling errors you've found have already been fixed. Once the document has been published this repository will along with the rest of the organisation be archived and handed over as per RFC 8875.

@fiestajetsam I don't see a requirement to archive this repo and organisation in RFC 8875. Please could you provide a reference?

@cabo
Copy link
Member

cabo commented Feb 22, 2024

My plan was to complete the auth48 branch and then to backport the XML changes to the .md, so there is a solid base for a potential future -bis. (Work won't actually happen on something like this until there is consensus IETF should work on specific items.). Won't get to this before April, probably.

@glyn
Copy link
Collaborator

glyn commented Feb 22, 2024

It may be have premature to close the PR. Let's leave it open for consideration.

@glyn glyn reopened this Feb 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants