Skip to content

fix fastfetch wrapper#172

Merged
zimward merged 1 commit into
Lassulus:mainfrom
mun-luna:patch-1
Jul 11, 2026
Merged

fix fastfetch wrapper#172
zimward merged 1 commit into
Lassulus:mainfrom
mun-luna:patch-1

Conversation

@mun-luna

@mun-luna mun-luna commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

fastfetch claims it "couldnt find config" when no extension (.json or .jsonc) is on the config file, which stops this wrapper from working

@zimward zimward left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, thanks for the fix

@zimward

zimward commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

could you rename the commit to modules/fastfetch: add .jsonc extension to config file ? then i would merge it

fastfetch claims it "couldnt find config" when no extension (.json or .jsonc) is on the config file, which stops this wrapper from working
@mun-luna

Copy link
Copy Markdown
Contributor Author

renamed it !!

@zimward zimward merged commit b859260 into Lassulus:main Jul 11, 2026
2 checks passed
@mun-luna mun-luna deleted the patch-1 branch July 12, 2026 16:16
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