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

Adding mario keyboard #744

Open
wants to merge 7 commits into
base: vial
Choose a base branch
from
Open

Adding mario keyboard #744

wants to merge 7 commits into from

Conversation

YkieWang
Copy link

@YkieWang YkieWang commented Jun 6, 2024

Adding a keyboard named mario, which has a encoder in the left half and a joystick in the right half.

Signed-off-by: Yuting Wang <[email protected]>
@xyzz
Copy link
Contributor

xyzz commented Jun 23, 2024

failing to build due to duplicate uid

Signed-off-by: Yuting Wang <[email protected]>
@YkieWang
Copy link
Author

failing to build due to duplicate uid

I have revised it, thanks.

@xyzz
Copy link
Contributor

xyzz commented Jun 24, 2024

still failing to build

@YkieWang
Copy link
Author

still failing to build

I have fixed it, many thanks!

This is my first time uploading a keymap, and I encountered a few issues that resulted in a building failure. I've documented the problems I faced and the solutions I found to help others who might need it.

Add default keymap: Make sure you have a default keymap, because the first step in the workflow is to compile your default keymap.

Enable Actions on Your Cloned Branch: Make sure to enable GitHub Actions on your cloned branch. This way, every time you commit code, the workflow will automatically execute, allowing you to see the results and understand the causes of any issues.

Firmware Size Limit: The firmware size can easily exceed the maximum allowed size. To mitigate this, you can disable RGB features and other unnecessary functionalities.

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.

None yet

2 participants