CI: fix style checking by d-a-v · Pull Request #7222 · esp8266/Arduino
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 13.2k
Merged
devyte merged 3 commits intoesp8266:masterfrom
devyte merged 3 commits intoesp8266:masterfrom
Conversation
Copy link
Collaborator
d-a-v
commented
Apr 17, 2020
d-a-v
commented
- (some) core files are astyled
- ci was only checking
libraries/
=> now everything is checked
d-a-v added 2 commits
April 17, 2020 23:11- (some) core files are astyled - ci was only checking libraries => now everything is checked
devyte approved these changes Apr 17, 2020
Copy link
Collaborator
devyte
left a comment
devyte
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM at a glance
devyte
merged commit
ce0e63f
into
esp8266:master
earlephilhower approved these changes Apr 18, 2020
Copy link
Collaborator
earlephilhower
left a comment
earlephilhower
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
devyte
added this to the
2.7.0 milestone
Copy link
Collaborator
earlephilhower
commented
Apr 18, 2020
earlephilhower commented
Apr 18, 2020Missed it by that much. 😆
devyte
added
component: core
type: code cleanup
labels
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment