Skip to content

Reset grumpyscreen config when needed - #260

Merged
jamesturton merged 3 commits into
mainfrom
james/reset-grumpyscreen-config
Jul 29, 2026
Merged

Reset grumpyscreen config when needed#260
jamesturton merged 3 commits into
mainfrom
james/reset-grumpyscreen-config

Conversation

@jamesturton

Copy link
Copy Markdown
Collaborator

Follows the logic from the klipper config reset, except it removed the config from the overlay rather than commenting the modifications out.
Maybe at some point we should also use CONFIG_OVERRIDE_FILE, but that seems a little bit overkill right now

@jamesturton jamesturton added this to the 0.0.8 milestone Jul 19, 2026
@github-actions

Copy link
Copy Markdown

✅ Build Artifacts

Branch: james/reset-grumpyscreen-config
Build: 765ab64 (merge into main)

Artifact Size
CC1 Firmware 91.31 MB

View workflow run

@jamesturton
jamesturton force-pushed the james/reset-grumpyscreen-config branch from 7262b1f to eaf388a Compare July 19, 2026 22:12
@github-actions

Copy link
Copy Markdown

✅ Build Artifacts

Branch: james/reset-grumpyscreen-config
Build: eaf388a (merge into main)

Artifact Size
CC1 Firmware 91.31 MB

View workflow run

@suchmememanyskill suchmememanyskill left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code looks overall fine, have not tested it personally

@jamesturton

Copy link
Copy Markdown
Collaborator Author

Tested working on my machine

@jamesturton
jamesturton enabled auto-merge (squash) July 29, 2026 21:38
@jamesturton
jamesturton merged commit a962796 into main Jul 29, 2026
4 checks passed
@jamesturton
jamesturton deleted the james/reset-grumpyscreen-config branch July 29, 2026 21:40
@github-actions

Copy link
Copy Markdown

✅ Build Artifacts

Branch: james/reset-grumpyscreen-config
Build: 214a14a (merge into main)

Artifact Size
CC1 Firmware 91.31 MB

View workflow run

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.

2 participants