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

Remove own Stringer #80

Closed
rwxrob opened this issue Jun 19, 2021 · 0 comments
Closed

Remove own Stringer #80

rwxrob opened this issue Jun 19, 2021 · 0 comments

Comments

@rwxrob
Copy link
Owner

rwxrob commented Jun 19, 2021

Providing for either a string or a function that provides a dynamic
string is "cute" and as Ryan Dahl says in "10 Things I Regret About
Node" it's those cute things that you end up regretting in the end.

I already regret it. I had to reimplement all of fmt to support the
flattened versions. That was probably stupid. Then I can to implement
MarshalJSON and MarshYAML (and family) because I'm not just dealing
with basic JSON and YAML types. That is unnecessary and just does not
warrant possible value of having dynamic documentation.

Plus, this inhibits the greater good of bundling multi-language
documentation with CmdBox composite commands, which will never have any
function definitions in them. Having that is far more important than
supporting the possible use of dynamic documentation detection.

Besides, if a Command author wants to provide some insight into the
system it is far better to create a command for that than to build it
into some form of dynamic docs.

This is gonna hurt. I put a lot of time into this. Wasted time. But
sunk-cost fallacy be damned.

@rwxrob rwxrob closed this as completed Jun 20, 2021
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

No branches or pull requests

1 participant