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

Noodlepad Additions and Updates #22701

Merged

Conversation

jessel92
Copy link
Contributor

@jessel92 jessel92 commented Dec 18, 2023

Modified this keyboard according to failed PR from the previous day with help from @sigprof

Description

Renaming themadnoodle/noodlepadto themadnoodle/noodlepad/v1 and added themadnoodle/noodlepad/v2
Updated the formatting to the current JSON QMK formatting to the best of my abilities.
Cleaned up keymaps and readmes

Got a HUGE amount of help from @sigprof via QMK Discord, so thank you so much!

I also have other PRs Open for:

Adding Keyboard The Mad Noodle Udon13 #22704
Added Keyboard Noodlepad_Micro #22703

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

Modified this keyboard according to failed PR from the previous day with help from @sigprof
@github-actions github-actions bot added keyboard keymap via Adds via keymap and/or updates keyboard for via support labels Dec 18, 2023
keyboards/themadnoodle/noodlepad/config.h Outdated Show resolved Hide resolved
keyboards/themadnoodle/noodlepad/v1/readme.md Outdated Show resolved Hide resolved
keyboards/themadnoodle/noodlepad/v2/info.json Outdated Show resolved Hide resolved
keyboards/themadnoodle/noodlepad/v1/info.json Outdated Show resolved Hide resolved
keyboards/themadnoodle/noodlepad/v2/readme.md Outdated Show resolved Hide resolved
keyboards/themadnoodle/noodlepad/info.json Outdated Show resolved Hide resolved
jessel92 and others added 5 commits December 18, 2023 18:58
Co-authored-by: jack <0x6a73@protonmail.com>
Co-authored-by: jack <0x6a73@protonmail.com>
Co-authored-by: jack <0x6a73@protonmail.com>
Co-authored-by: jack <0x6a73@protonmail.com>
Copy link
Contributor Author

@jessel92 jessel92 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All should be good to go

@jessel92 jessel92 requested a review from waffle87 December 19, 2023 02:28
@zvecr
Copy link
Member

zvecr commented Dec 19, 2023

All should be good to go

image

Nope, thats a hard blocker on the PR being mergable.

@jessel92
Copy link
Contributor Author

All should be good to go

image

Nope, thats a hard blocker on the PR being mergable.

What does that mean? What caused it?

@zvecr
Copy link
Member

zvecr commented Dec 19, 2023

lib changes in fb05cce

@adophoxia
Copy link
Contributor

adophoxia commented Dec 19, 2023

There is lib/chibios-contrib submodule changes in this PR. Most likely snuck in while you were making changes previously.

@jessel92
Copy link
Contributor Author

jessel92 commented Dec 19, 2023

There is lib/chibios-contrib submodule changes in this PR. Most likely snuck in while you were making changes previously.

@adophoxia Well crap, can I do something about it???

@jessel92
Copy link
Contributor Author

@zvecr @adophoxia I reverted the commit. Should that fix it?

This reverts commit 81dfa68.
@jessel92
Copy link
Contributor Author

@zvecr @adophoxia I reverted the commit. Should that fix it?

@zvecr @adophoxia IT FIXED IT! We should be good now right?

Co-authored-by: jack <0x6a73@protonmail.com>
@waffle87 waffle87 requested a review from a team December 19, 2023 20:58
keyboards/themadnoodle/noodlepad/v2/info.json Outdated Show resolved Hide resolved
keyboards/themadnoodle/noodlepad/v1/readme.md Outdated Show resolved Hide resolved
keyboards/themadnoodle/noodlepad/v2/readme.md Outdated Show resolved Hide resolved
keyboards/themadnoodle/noodlepad/keymaps/via/rules.mk Outdated Show resolved Hide resolved
keyboards/themadnoodle/noodlepad/keymaps/default/rules.mk Outdated Show resolved Hide resolved
jessel92 and others added 5 commits December 19, 2023 21:56
Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
@drashna drashna merged commit 7e47446 into qmk:develop Dec 23, 2023
3 checks passed
@jessel92 jessel92 deleted the Noodlepad-additions-and-updates-231218 branch December 27, 2023 04:31
itsjonny96 pushed a commit to itsjonny96/qmk_firmware that referenced this pull request Jan 7, 2024
Co-authored-by: jack <0x6a73@protonmail.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
mechlovin pushed a commit to mechlovin/qmk_firmware that referenced this pull request Jan 17, 2024
Co-authored-by: jack <0x6a73@protonmail.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
mechlovin pushed a commit to mechlovin/qmk_firmware that referenced this pull request Jan 19, 2024
Co-authored-by: jack <0x6a73@protonmail.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
nuess0r pushed a commit to nuess0r/qmk_firmware that referenced this pull request Sep 8, 2024
Co-authored-by: jack <0x6a73@protonmail.com>
Co-authored-by: Ryan <fauxpark@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
keyboard keymap via Adds via keymap and/or updates keyboard for via support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants