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

Lily58 default keymap fix / add keymap #4190

Merged
merged 4 commits into from
Oct 22, 2018
Merged

Lily58 default keymap fix / add keymap #4190

merged 4 commits into from
Oct 22, 2018

Conversation

kata0510
Copy link
Contributor

No description provided.

*/
#pragma once

#include "config.h"
Copy link
Member

Choose a reason for hiding this comment

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

This line is not actually needed, and should be removed.

Specifically, the build script automatically includes all of the config.h files, so this is not necessary.




void persistent_default_layer_set(uint16_t default_layer) {
Copy link
Member

Choose a reason for hiding this comment

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

Could you remove this function?

It's been depreciated in favor of the set_single_persistent_default_layer function (which is a core function).

https://docs.qmk.fm/#/ref_functions?id=setting-the-persistent-default-layer

@@ -1,9 +1,7 @@
SRC += matrix.c \
i2c.c \
split_util.c \
serial.c \
ssd1306.c
Copy link
Member

Choose a reason for hiding this comment

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

Is there a reason for removing this?

@@ -1,330 +0,0 @@
#ifdef SSD1306OLED
Copy link
Member

Choose a reason for hiding this comment

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

Was there a reason for removing this?

@kata0510
Copy link
Contributor Author

@drashna
Thank you for review
Fixed the file
Deletion of ssd1306 decided not to put OLED module on Lily58.

@drashna
Copy link
Member

drashna commented Oct 22, 2018

Okay. Thanks for the clarification.

@drashna drashna merged commit f69333a into qmk:master Oct 22, 2018
zer09 pushed a commit to zer09/qmk_firmware that referenced this pull request Nov 3, 2018
* keymap fix/add

* keymap fix

* file fix

* typo fix
3n4rKy pushed a commit to 3n4rKy/qmk_firmware that referenced this pull request Nov 4, 2018
* keymap fix/add

* keymap fix

* file fix

* typo fix
@kata0510 kata0510 deleted the lily58_keymap_fix branch December 23, 2018 10:50
rseymour pushed a commit to rseymour/qmk_firmware that referenced this pull request Mar 13, 2019
* keymap fix/add

* keymap fix

* file fix

* typo fix
yamad pushed a commit to yamad/qmk_firmware that referenced this pull request Apr 10, 2019
* keymap fix/add

* keymap fix

* file fix

* typo fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants