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

Add Funky40 support #8942

Merged
merged 14 commits into from
Apr 27, 2020
Merged

Add Funky40 support #8942

merged 14 commits into from
Apr 27, 2020

Conversation

TheFourthCow
Copy link
Contributor

@TheFourthCow TheFourthCow commented Apr 27, 2020

Adding files for the Funky40 a new board from /u/TheFourthCow.

Edited code to ensure compilation is successful, removed a recommendation from a previous pull request as the changing of #include "funky40.h" to what was recommended resulted in many errors, unsure of what to do if this does not compile correctly when this is submitted, will seek assistance in the discord tomorrow. Thanks again!

convinced keymap.c and funky40.h to work with KEYMAP changed to LAYOUT as requested.

  • 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.
  • 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).

TheFourthCow and others added 4 commits April 26, 2020 16:31
This should add all proper files for the funky40 a keyboard I designed for myself, /u/TheFourthcow, a 40% ortholinear with split spacebar.
…l reccomended changes from my previous pull request
…ide with no errors hopefully this one works!
@fauxpark
Copy link
Member

You do not need to keep opening new PRs. Just commit and push to the same branch and the original PR will be updated. If you've accepted suggestions on the PR, you will need to pull those changes to your local copy first.

keyboards/funky40/readme.mk.mk Outdated Show resolved Hide resolved
keyboards/funky40/keymaps/default/readme.md.md Outdated Show resolved Hide resolved
keyboards/funky40/readme.mk.mk Outdated Show resolved Hide resolved
keyboards/funky40/keymaps/default/keymap.c Outdated Show resolved Hide resolved
keyboards/funky40/keymaps/default/keymap.c Outdated Show resolved Hide resolved
keyboards/funky40/keymaps/default/keymap.c Outdated Show resolved Hide resolved
keyboards/funky40/config.h Outdated Show resolved Hide resolved
@fauxpark fauxpark requested a review from a team April 27, 2020 04:46
@zvecr zvecr changed the title Funky40 file update with revisions should compile correctly Add Funky40 support Apr 27, 2020
keyboards/funky40/config.h Outdated Show resolved Hide resolved
keyboards/funky40/readme.md Outdated Show resolved Hide resolved
keyboards/funky40/readme.md Outdated Show resolved Hide resolved
Copy link
Member

@noroadsleft noroadsleft 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 me.

@fauxpark
Copy link
Member

Just missing an info.json.

@noroadsleft noroadsleft merged commit 4c51049 into qmk:master Apr 27, 2020
@noroadsleft
Copy link
Member

Thanks!

I'll get the info.json later.

@TheFourthCow
Copy link
Contributor Author

do I need to get the info.json? I'm not sure how. Thanks for reviewing my files!

@noroadsleft
Copy link
Member

do I need to get the info.json? I'm not sure how. Thanks for reviewing my files!

@TheFourthCow If you want to do it yourself, here's how. If you need any help just ping me on this PR or a new one. I'm also on QMK Discord, with this same username.

@TheFourthCow TheFourthCow deleted the funky branch April 28, 2020 00:41
@TheFourthCow TheFourthCow restored the funky branch April 28, 2020 00:43
mrlinuxfish pushed a commit to mrlinuxfish/qmk_firmware that referenced this pull request May 3, 2020
* Adding all relevant files for the Funky40

This should add all proper files for the funky40 a keyboard I designed for myself, /u/TheFourthcow, a 40% ortholinear with split spacebar.

* Second attempt to add all relevant files for the funky40, includes all reccomended changes from my previous pull request

* Revised most files for Funky40 including reccomenations from my previous pull request

* further modifications made to default funky40 board, compiles on my side with no errors hopefully this one works!

* Update keyboards/funky40/readme.mk.mk

* Update keyboards/funky40/keymaps/default/readme.md.md

* Update keyboards/funky40/keymaps/default/keymap.c

* Update keyboards/funky40/keymaps/default/keymap.c

* Update keyboards/funky40/keymaps/default/keymap.c

* Update keyboards/funky40/config.h

* updating readmes and keymap

* final update to keymap and readmes should function correctly with updates requested

* made changes as requested by noroadsleft to config and readme
bitherder pushed a commit to bitherder/qmk_firmware that referenced this pull request May 15, 2020
* Adding all relevant files for the Funky40

This should add all proper files for the funky40 a keyboard I designed for myself, /u/TheFourthcow, a 40% ortholinear with split spacebar.

* Second attempt to add all relevant files for the funky40, includes all reccomended changes from my previous pull request

* Revised most files for Funky40 including reccomenations from my previous pull request

* further modifications made to default funky40 board, compiles on my side with no errors hopefully this one works!

* Update keyboards/funky40/readme.mk.mk

* Update keyboards/funky40/keymaps/default/readme.md.md

* Update keyboards/funky40/keymaps/default/keymap.c

* Update keyboards/funky40/keymaps/default/keymap.c

* Update keyboards/funky40/keymaps/default/keymap.c

* Update keyboards/funky40/config.h

* updating readmes and keymap

* final update to keymap and readmes should function correctly with updates requested

* made changes as requested by noroadsleft to config and readme
sowbug pushed a commit to sowbug/qmk_firmware that referenced this pull request May 24, 2020
* Adding all relevant files for the Funky40

This should add all proper files for the funky40 a keyboard I designed for myself, /u/TheFourthcow, a 40% ortholinear with split spacebar.

* Second attempt to add all relevant files for the funky40, includes all reccomended changes from my previous pull request

* Revised most files for Funky40 including reccomenations from my previous pull request

* further modifications made to default funky40 board, compiles on my side with no errors hopefully this one works!

* Update keyboards/funky40/readme.mk.mk

* Update keyboards/funky40/keymaps/default/readme.md.md

* Update keyboards/funky40/keymaps/default/keymap.c

* Update keyboards/funky40/keymaps/default/keymap.c

* Update keyboards/funky40/keymaps/default/keymap.c

* Update keyboards/funky40/config.h

* updating readmes and keymap

* final update to keymap and readmes should function correctly with updates requested

* made changes as requested by noroadsleft to config and readme
turky pushed a commit to turky/qmk_firmware that referenced this pull request Jun 13, 2020
* Adding all relevant files for the Funky40

This should add all proper files for the funky40 a keyboard I designed for myself, /u/TheFourthcow, a 40% ortholinear with split spacebar.

* Second attempt to add all relevant files for the funky40, includes all reccomended changes from my previous pull request

* Revised most files for Funky40 including reccomenations from my previous pull request

* further modifications made to default funky40 board, compiles on my side with no errors hopefully this one works!

* Update keyboards/funky40/readme.mk.mk

* Update keyboards/funky40/keymaps/default/readme.md.md

* Update keyboards/funky40/keymaps/default/keymap.c

* Update keyboards/funky40/keymaps/default/keymap.c

* Update keyboards/funky40/keymaps/default/keymap.c

* Update keyboards/funky40/config.h

* updating readmes and keymap

* final update to keymap and readmes should function correctly with updates requested

* made changes as requested by noroadsleft to config and readme
jakobaa pushed a commit to jakobaa/qmk_firmware that referenced this pull request Jul 7, 2020
* Adding all relevant files for the Funky40

This should add all proper files for the funky40 a keyboard I designed for myself, /u/TheFourthcow, a 40% ortholinear with split spacebar.

* Second attempt to add all relevant files for the funky40, includes all reccomended changes from my previous pull request

* Revised most files for Funky40 including reccomenations from my previous pull request

* further modifications made to default funky40 board, compiles on my side with no errors hopefully this one works!

* Update keyboards/funky40/readme.mk.mk

* Update keyboards/funky40/keymaps/default/readme.md.md

* Update keyboards/funky40/keymaps/default/keymap.c

* Update keyboards/funky40/keymaps/default/keymap.c

* Update keyboards/funky40/keymaps/default/keymap.c

* Update keyboards/funky40/config.h

* updating readmes and keymap

* final update to keymap and readmes should function correctly with updates requested

* made changes as requested by noroadsleft to config and readme
sjmacneil pushed a commit to sjmacneil/qmk_firmware that referenced this pull request Feb 19, 2021
* Adding all relevant files for the Funky40

This should add all proper files for the funky40 a keyboard I designed for myself, /u/TheFourthcow, a 40% ortholinear with split spacebar.

* Second attempt to add all relevant files for the funky40, includes all reccomended changes from my previous pull request

* Revised most files for Funky40 including reccomenations from my previous pull request

* further modifications made to default funky40 board, compiles on my side with no errors hopefully this one works!

* Update keyboards/funky40/readme.mk.mk

* Update keyboards/funky40/keymaps/default/readme.md.md

* Update keyboards/funky40/keymaps/default/keymap.c

* Update keyboards/funky40/keymaps/default/keymap.c

* Update keyboards/funky40/keymaps/default/keymap.c

* Update keyboards/funky40/config.h

* updating readmes and keymap

* final update to keymap and readmes should function correctly with updates requested

* made changes as requested by noroadsleft to config and readme
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.

4 participants