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: make header color configurable #22

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jderuiter
Copy link

The header template (TODOFISH_HEADER) was parsed before the config file was sourced, so the default color was always used.

@hugokernel
Copy link
Owner

If you want to override the config like TODOFISH_HEADER, you need to use the -c argument and the config file will be loaded after.

@jderuiter
Copy link
Author

Actually I didn't want to override TODOFISH_HEADER, but I wanted it to respect the COLOR_TITLE that I set in my config. It kept being bright green, despite setting COLOR_TITLE to something else. If TODOFISH_HEADER should also be configurable I will need to add an additional check to see whether it was overridden in by the config file.

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