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

Docs: Bring coding files up to style and add license #1314

Merged
merged 9 commits into from
Aug 25, 2022

Conversation

jwrober
Copy link
Collaborator

@jwrober jwrober commented Aug 24, 2022

No description provided.

@jwrober jwrober requested a review from daavko August 24, 2022 15:40
Copy link
Collaborator

@daavko daavko left a comment

Choose a reason for hiding this comment

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

Too many occurences to go comment on individually: double spaces after periods. We don't use them in any of the newly written documentation, but it looks like whoever was writing the hacking guide originally was fond of them. Not sure whether it's worth it to add this to the style guide, or to just keep it in mind when bringing docs up to style.

docs/Coding/agents.rst Outdated Show resolved Hide resolved
docs/Coding/agents.rst Outdated Show resolved Hide resolved
docs/Coding/agents.rst Outdated Show resolved Hide resolved
docs/Coding/agents.rst Outdated Show resolved Hide resolved
docs/Coding/ai.rst Outdated Show resolved Hide resolved
docs/Coding/hacking.rst Outdated Show resolved Hide resolved
docs/Coding/hacking.rst Outdated Show resolved Hide resolved
docs/Coding/hacking.rst Outdated Show resolved Hide resolved
docs/Coding/hacking.rst Outdated Show resolved Hide resolved
docs/Coding/hacking.rst Outdated Show resolved Hide resolved
@jwrober jwrober enabled auto-merge (rebase) August 25, 2022 21:20
@jwrober jwrober merged commit c678ca2 into longturn:master Aug 25, 2022
@jwrober jwrober deleted the docs/coding branch September 12, 2022 17:57
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.

2 participants