Skip to content

Fix small typo in required-knowledge.md#287

Merged
benjie merged 1 commit intographile:developfrom
morleytatro:patch-3
Oct 18, 2021
Merged

Fix small typo in required-knowledge.md#287
benjie merged 1 commit intographile:developfrom
morleytatro:patch-3

Conversation

@morleytatro
Copy link
Copy Markdown
Contributor

@morleytatro morleytatro commented Oct 17, 2021

small typo

Description

Performance impact

Security impact

Checklist

  • My code matches the project's code style and yarn lint:fix passes.
  • I've added tests for the new feature, and yarn test passes.
  • I have detailed the new feature in the relevant documentation.
  • I have added this feature to 'Pending' in the RELEASE_NOTES.md file (if one exists).
  • If this is a breaking change I've explained why.

benjie
benjie approved these changes Oct 18, 2021
View reviewed changes
Copy link
Copy Markdown
Member

@benjie benjie left a comment

Choose a reason for hiding this comment

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

Good catch! 🙌

@benjie benjie changed the title Update required-knowledge.md Fix small typo in required-knowledge.md Oct 18, 2021
@benjie benjie merged commit c986c01 into graphile:develop Oct 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@benjie benjie benjie approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@morleytatro @benjie