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

Created Southpole keyboard folder #4946

Merged
merged 17 commits into from
Jan 28, 2019
Merged

Created Southpole keyboard folder #4946

merged 17 commits into from
Jan 28, 2019

Conversation

ericduns
Copy link
Contributor

@ericduns ericduns commented Jan 26, 2019

Description

Added a folder for the southpole keyboard created by u/waxpoetic. The folder includes all necessary files as well as a modified keymap file.

Types of changes

  • Core
  • Bugfix
  • New Feature
  • Enhancement/Optimization
  • Keyboard (addition or update)
  • Keymap/Layout/Userspace (addition or update)
  • Documentation

Issues Fixed or Closed by this PR

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. (https://docs.qmk.fm/#/contributing)
  • 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).

added southpole keyboard folder and files including keymaps folder. Keymaps have errors and will not compile yet
added manufacturer and product name and description
changed formating added layer
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.

In addition to the comments below, the keyboard should have a readme file at keyboards/southpole/readme.md.

See quantum/template/avr/readme.md for the template.

@ericduns
Copy link
Contributor Author

Ok, I'm a n00b (easy to tell I'm sure) when it comes to qmk, but since making the suggested changes I've broke everything lol. Everything was compiling before, now I got 99 errors and not 1 .hex file. Please guide me lol.

@drashna
Copy link
Member

drashna commented Jan 28, 2019

So, this is a limited, one time run board, or?

Eg, if someone wanted this. how would they get one, aside from /r/mm?

@ericduns
Copy link
Contributor Author

ericduns commented Jan 28, 2019 via email

@drashna
Copy link
Member

drashna commented Jan 28, 2019

Thanks!

@drashna drashna merged commit e0e8097 into qmk:master Jan 28, 2019
zer09 pushed a commit to zer09/qmk_firmware that referenced this pull request Feb 9, 2019
* Added southpole keyboard

added southpole keyboard folder and files including keymaps folder. Keymaps have errors and will not compile yet

* Add files via upload

added manufacturer and product name and description

* Update config.h

* Add files via upload

changed formating added layer

* cleaned up files, fixed foobeard keymap, and added maker info

* fixed keymap files

* Update keyboards/southpole/keymaps/foobeard/keymap.c

Co-Authored-By: ericduns <[email protected]>

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

Co-Authored-By: ericduns <[email protected]>

* made changes fixing outdated lines and added readme.md

* made requested changes for pr to all files

* removed vs code and added new line to southpole.h
dlhextall pushed a commit to dlhextall/qmk_firmware that referenced this pull request May 24, 2019
* Added southpole keyboard

added southpole keyboard folder and files including keymaps folder. Keymaps have errors and will not compile yet

* Add files via upload

added manufacturer and product name and description

* Update config.h

* Add files via upload

changed formating added layer

* cleaned up files, fixed foobeard keymap, and added maker info

* fixed keymap files

* Update keyboards/southpole/keymaps/foobeard/keymap.c

Co-Authored-By: ericduns <[email protected]>

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

Co-Authored-By: ericduns <[email protected]>

* made changes fixing outdated lines and added readme.md

* made requested changes for pr to all files

* removed vs code and added new line to southpole.h
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