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

fix(mapper): remove unnecessary globals #529

Merged
merged 3 commits into from
Jul 4, 2023
Merged

Conversation

LordMZTE
Copy link
Contributor

@LordMZTE LordMZTE commented Jul 1, 2023

This uses Lua's environment features in order to modify global scope for loading hlgroups instead of the old hacky solution which modified _G.

@rewhile
Copy link
Contributor

rewhile commented Jul 2, 2023

This PR is creating side effect of random flavour during startup, still no idea why. DO NOT MERGE

@mrtnvgr mrtnvgr marked this pull request as draft July 2, 2023 05:58
@rewhile rewhile marked this pull request as ready for review July 3, 2023 01:43
@rewhile rewhile changed the title fix(mapper): use environments instead of modifying _G fix(mapper): remove unnecessary globals Jul 4, 2023
@rewhile rewhile merged commit c75562c into catppuccin:main Jul 4, 2023
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