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

add metadata toml fields for wezterm #184

Merged
merged 8 commits into from
Jul 20, 2022
Merged

Conversation

digitallyserviced
Copy link
Contributor

This is for #183 #156 and for wezterm to properly parse the metadata content for nightfox related toml tempmlates.

also update comment referring to cmd to run to generate the templates

Copy link
Owner

@EdenEast EdenEast left a comment

Choose a reason for hiding this comment

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

Thanks for taking this on.

lua/nightfox/extra/wezterm.lua Outdated Show resolved Hide resolved
lua/nightfox/extra/wezterm.lua Outdated Show resolved Hide resolved
digitallyserviced and others added 6 commits July 19, 2022 23:10
Co-authored-by: Wez Furlong <wez@wezfurlong.org>
Co-authored-by: EdenEast <edenofest@gmail.com>
Co-authored-by: Wez Furlong <wez@wezfurlong.org>
Co-authored-by: EdenEast <edenofest@gmail.com>
Co-authored-by: Wez Furlong <wez@wezfurlong.org>
@digitallyserviced
Copy link
Contributor Author

Hopefully all is good now?

Updated tpl as per sugg's, and precompiles.

@EdenEast
Copy link
Owner

LGTM

@EdenEast EdenEast merged commit f8e1a09 into EdenEast:main Jul 20, 2022
wez added a commit to wez/wezterm that referenced this pull request Jul 20, 2022
note that "_bash (Gogh)" is removed by this as it is actually
a duplicate of nightfox but with a different name, according
to my dedup logic.

I may add a way to express aliases as a followup.

refs: EdenEast/nightfox.nvim#184
refs: EdenEast/nightfox.nvim#183
refs: EdenEast/nightfox.nvim#156
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.

3 participants