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

Formatter file name should be user-configurable. #737

Open
ShaneMcC opened this Issue Dec 31, 2016 · 2 comments

Comments

Projects
None yet
2 participants
@ShaneMcC
Member

ShaneMcC commented Dec 31, 2016

Since the formatters were redone ages ago we only look in a single location for formatters now.

Previously the general.formatters setting was parsed and the formatters read from here instead which was a bit more flexible.

@csmith

This comment has been minimized.

Show comment
Hide comment
@csmith

csmith Jan 11, 2017

Member

Not sure I understand the point... Why would you have more than one formatter? Why is it more flexible to run the client, change a setting, and reload the formatter than to just move the files on disk (or use a different profile directory with the -d flag)?

Member

csmith commented Jan 11, 2017

Not sure I understand the point... Why would you have more than one formatter? Why is it more flexible to run the client, change a setting, and reload the formatter than to just move the files on disk (or use a different profile directory with the -d flag)?

@ShaneMcC

This comment has been minimized.

Show comment
Hide comment
@ShaneMcC

ShaneMcC Jan 13, 2017

Member

Meant I could change the formatter by using /set general formatters foo.format and then after reload it would use that rather than worrying about accidentally losing the formatter when moving things around which was a bit safer.

It's not essential, it was just a nice-to-have.

Member

ShaneMcC commented Jan 13, 2017

Meant I could change the formatter by using /set general formatters foo.format and then after reload it would use that rather than worrying about accidentally losing the formatter when moving things around which was a bit safer.

It's not essential, it was just a nice-to-have.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment