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

Helix keyboard: Move definition place of LAYOUT_kc macro #3333

Merged
merged 1 commit into from
Jul 8, 2018

Conversation

epaew
Copy link
Contributor

@epaew epaew commented Jul 7, 2018

No description provided.

@drashna
Copy link
Member

drashna commented Jul 7, 2018

@MakotoKurauchi Is this okay?

It looks right to me, but I'd like your OK before merging.

@MakotoKurauchi
Copy link
Contributor

It is OK. No problem.
Thank you very much.

@drashna
Copy link
Member

drashna commented Jul 8, 2018

Thanks!

@drashna drashna merged commit ebe4c3d into qmk:master Jul 8, 2018
@epaew epaew deleted the fix_helixs_layout_kc branch July 8, 2018 04:23
alexey-danilov pushed a commit to alexey-danilov/qmk_firmware that referenced this pull request Jul 27, 2018
ChrissiQ pushed a commit to ChrissiQ/qmk_firmware that referenced this pull request Sep 25, 2018
yamad pushed a commit to yamad/qmk_firmware that referenced this pull request Apr 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants