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

Minor Documentation Tweaks #904

Merged
merged 6 commits into from
Nov 1, 2019
Merged

Conversation

cfilby
Copy link

@cfilby cfilby commented Oct 16, 2019

Cleans up some formatting and typos in the docs. Tries to make code samples more consistently highlighted/formatted.

I have:

  • [*] Added tests covering the bug / feature (see testing)
  • [*] Updated any relevant documentation (see docs)

@coveralls
Copy link

coveralls commented Oct 16, 2019

Coverage Status

Coverage remained the same at 61.95% when pulling 5eb6bef on cfilby:fix-config-docs into dbc8842 on 99designs:master.

@cfilby cfilby changed the title Fix documentation bug Minor Documentation Tweaks Oct 16, 2019
@cfilby
Copy link
Author

cfilby commented Oct 25, 2019

Anything else you need from me to get this merged?

README.md Outdated
id: ID!
name: String!
friends: [User!]!
id: ID!
Copy link
Collaborator

Choose a reason for hiding this comment

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

what happened to the indenting here?

Copy link
Author

Choose a reason for hiding this comment

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

I went ahead and tried to fix the indentation for a few README snippets. VSCode got a little crazy with the auto formatting.

Let me know if there's anything else you see that needs tweaking.

@vektah vektah merged commit 1172128 into 99designs:master Nov 1, 2019
cgxxv pushed a commit to cgxxv/gqlgen that referenced this pull request Mar 25, 2022
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.

4 participants