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

Tradestation code tidy and readme refactor #4784

Merged
merged 3 commits into from
Jan 5, 2019

Conversation

noroadsleft
Copy link
Member

Commit Log

Tradestation: tidy code (c079a44)

  • correct JSON syntax on info.json
  • visual key alignment on info.json
  • white space changes on rules.mk and tradestation.h (readability/QMK conventions)

Tradestation: readme refactor (ee5b0af)

Modify readme.md to more closely match QMK template.


Notes

@flehrad, the link in the readme for libsyn.com is "error 404" but I left it in in case you can provide me a good link. I searched but didn't find one.

- correct JSON syntax on info.json
- visual key alignment on info.json
- white space changes on rules.mk and tradestation.h (readability/QMK conventions)
Modify readme.md to more closely match QMK template.
@flehrad
Copy link
Contributor

flehrad commented Jan 5, 2019

thank you so much for doing this.
Looks like they've changed things on libsyn - http://theboard.libsyn.com/ should be the correct one, however in keeping in line with QMK template, you can just get rid of all of those links if you like.

Update The Board podcast link for libsyn.
@noroadsleft
Copy link
Member Author

The template is more of a starting point. You can include extra information, as you have. Some of the readme files actually have fairly detailed build logs.

@drashna
Copy link
Member

drashna commented Jan 5, 2019

Thanks!

@drashna drashna merged commit 27b512d into qmk:master Jan 5, 2019
@noroadsleft noroadsleft deleted the cf/hw_tradestation branch January 6, 2019 02:52
Shinichi-Ohki added a commit to Shinichi-Ohki/qmk_firmware that referenced this pull request Jan 10, 2019
* 'master' of https://github.com/qmk/qmk_firmware: (63 commits)
  Keymap: ave-63's iris layout (qmk#4812)
  Update bakingpy 4x12 keymap and add test mode for debugging/development (qmk#4810)
  Fix Mac mute keycode (qmk#4809)
  Add KBD75 keymap (qmk#4808)
  Fix pinout of split hand and LED, remove flip half option
  Tidy up Mod-Tap shortcuts (qmk#4806)
  Add missing parentheses to some important macros (qmk#4775)
  Keyboard: Downbubble refactor and Configurator fix (qmk#4798)
  Alternate keymap for Alpha keyboard, enjoy! (qmk#4797)
  Keymap: Added Model F-inspired layout for YMD[K]96 (qmk#4777)
  Improve consistency in UNICODEMAP code and docs, update docs/understanding_qmk (qmk#4774)
  Update to arm_atsam wait and timer routines
  Add Downbubble to Handwired repository (qmk#4794)
  Final HS60v2 changes. (qmk#4790)
  Keyboard: Fractal layout macro and readme cleanup (qmk#4789)
  Keymap: added my espectro keymap (qmk#4791)
  Keyboard: Numbrero: Configurator fix and code tidy (qmk#4787)
  Keyboard: Tradestation code tidy and readme refactor (qmk#4784)
  Keyboard: update readme with ps2avr flashing instructions (qmk#4776)
  add Pinky keyboard (qmk#4748)
  ...
rseymour pushed a commit to rseymour/qmk_firmware that referenced this pull request Mar 13, 2019
* Tradestation: tidy code

- correct JSON syntax on info.json
- visual key alignment on info.json
- white space changes on rules.mk and tradestation.h (readability/QMK conventions)

* Tradestation: readme refactor

Modify readme.md to more closely match QMK template.

* Tradestation: readme update

Update The Board podcast link for libsyn.
djthread pushed a commit to djthread/qmk_firmware that referenced this pull request Mar 17, 2019
* Tradestation: tidy code

- correct JSON syntax on info.json
- visual key alignment on info.json
- white space changes on rules.mk and tradestation.h (readability/QMK conventions)

* Tradestation: readme refactor

Modify readme.md to more closely match QMK template.

* Tradestation: readme update

Update The Board podcast link for libsyn.
dlhextall pushed a commit to dlhextall/qmk_firmware that referenced this pull request May 24, 2019
* Tradestation: tidy code

- correct JSON syntax on info.json
- visual key alignment on info.json
- white space changes on rules.mk and tradestation.h (readability/QMK conventions)

* Tradestation: readme refactor

Modify readme.md to more closely match QMK template.

* Tradestation: readme update

Update The Board podcast link for libsyn.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants