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

Improve documentation 1 to implement 1 #34

Merged
merged 10 commits into from
May 21, 2020

Conversation

schettn
Copy link
Member

@schettn schettn commented May 20, 2020

Checklist:

  • Have you added an explanation of what your changes do and why you'd like them to be included?
  • Have you updated or added documentation for the change?
  • Have you tested your changes with successful results?

Type of Changes:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Documentation (non-breaking change which adds documentation)
  • Breaking change (fix or feature that would cause existing functionality to change)

What is the current behavior? (link to any open issues here)

What is the new behavior (if this is a feature change)?

Other information:

Now when saving a typescript file it gets automatically formatted using prettier.
The documentation has been refined to fit the new coding style (snek-tonic). This includes setting new anchors, removing incorrect periods and some general adjustments.
Ref: https://github.com/snek-at/tonic/blob/master/STYLE_GUIDE.md
Spaces in anchors have been removed due to inconsistency with the snek-tonic.
A anchor has been added to the console.error
@schettn schettn added the documentation Improvements or additions to documentation label May 20, 2020
@schettn schettn requested review from Aichnerc and pinterid May 20, 2020 16:06
@schettn schettn self-assigned this May 20, 2020
@Aichnerc Aichnerc added this to Review in progress in SNEK Network for Engineers and Knowledged via automation May 20, 2020
Copy link
Member

@Aichnerc Aichnerc left a comment

Choose a reason for hiding this comment

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

🥐 As crooked as this 𝖈𝖗𝖔𝖎𝖘𝖘𝖆𝖓𝖙.

.vscode/settings.json Outdated Show resolved Hide resolved
.vscode/settings.json Outdated Show resolved Hide resolved
src/templates/snek/gql/tasks/general.ts Outdated Show resolved Hide resolved
src/templates/snek/gql/tasks/general.ts Outdated Show resolved Hide resolved
src/templates/snek/gql/tasks/index.ts Outdated Show resolved Hide resolved
@schettn schettn changed the title Improve documentation 1 Improve documentation 1 to implement 1 May 20, 2020
The documentation quality has been improved due to q request of @Aichnerc.
@schettn schettn requested a review from Aichnerc May 20, 2020 16:35
The comments have been removed because GitHub does not like it.
src/endpoints/index.ts Outdated Show resolved Hide resolved
src/endpoints/scraper.ts Outdated Show resolved Hide resolved
src/session/index.ts Outdated Show resolved Hide resolved
src/session/sessions.ts Outdated Show resolved Hide resolved
src/session/sessions.ts Outdated Show resolved Hide resolved
src/session/sessions.ts Outdated Show resolved Hide resolved
src/session/sessions.ts Outdated Show resolved Hide resolved
src/session/sessions.ts Outdated Show resolved Hide resolved
Copy link
Member

@pinterid pinterid left a comment

Choose a reason for hiding this comment

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

Looks good after some changes.

@pinterid
Copy link
Member

The type of change has to be set before merging!

The documentation quality has been improved due to a request of @pinterid.
@schettn schettn requested a review from pinterid May 21, 2020 08:32
Copy link
Member

@pinterid pinterid left a comment

Choose a reason for hiding this comment

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

Looks good after some changes.

The code quality has been improved due to a request of @pinterid.
@pinterid pinterid self-requested a review May 21, 2020 08:45
Copy link
Member

@pinterid pinterid left a comment

Choose a reason for hiding this comment

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

Looks good to merge.

Copy link
Member

@Aichnerc Aichnerc left a comment

Choose a reason for hiding this comment

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

LGTM 🥞

SNEK Network for Engineers and Knowledged automation moved this from Review in progress to Reviewer approved May 21, 2020
@schettn schettn requested a review from kleberbaum May 21, 2020 17:06
Any changes that affect the code have been undone.
Copy link
Member

@kleberbaum kleberbaum left a comment

Choose a reason for hiding this comment

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

Should be ok to merge after a few changes.

src/endpoints/apollo.ts Outdated Show resolved Hide resolved
src/endpoints/index.ts Outdated Show resolved Hide resolved
src/index.ts Show resolved Hide resolved
src/index.ts Outdated Show resolved Hide resolved
src/session/sessions.ts Show resolved Hide resolved
src/templates/snek/gql/tasks/auth.ts Outdated Show resolved Hide resolved
The documentation quality has been improved due to a request of @kleberbaum.
@kleberbaum kleberbaum self-requested a review May 21, 2020 18:15
Copy link
Member

@kleberbaum kleberbaum left a comment

Choose a reason for hiding this comment

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

Should be ok to merge.

@kleberbaum kleberbaum merged commit ddefded into implement-1 May 21, 2020
SNEK Network for Engineers and Knowledged automation moved this from Reviewer approved to Done May 21, 2020
@schettn schettn mentioned this pull request May 29, 2020
6 tasks
@schettn schettn added this to the v0.1 milestone May 29, 2020
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
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

The code should be adapted to the new coding guideline The anchors are not defined well
4 participants