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: fix code sample #16623

Merged
merged 1 commit into from
Mar 12, 2022
Merged

docs: fix code sample #16623

merged 1 commit into from
Mar 12, 2022

Conversation

Kriechi
Copy link
Contributor

@Kriechi Kriechi commented Mar 12, 2022

Description

This PR fixes a code sample in the docs for pointing devices.
There was a semicolon missing and I adopted the same style as in the code sample below (two assign statements instead of mangling them into one line).

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout/userspace (addition or update)
  • Documentation

Checklist

  • My code follows the code style of this project: C, Python
  • I have read the PR Checklist document and have made the appropriate changes.
  • [] My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

@fauxpark fauxpark requested a review from a team March 12, 2022 10:37
@zvecr zvecr merged commit 86b1231 into qmk:master Mar 12, 2022
@Kriechi Kriechi deleted the docs-pointing-code-fix branch March 12, 2022 19:49
tacahiroy pushed a commit to tacahiroy/qmk_firmware that referenced this pull request Mar 18, 2022
0xcharly pushed a commit to Bastardkb/bastardkb-qmk that referenced this pull request Jul 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants