fix: trin cli flag displays wrong version info #1609
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What was wrong?
#1581 (comment)
How was it fixed?
After a little research into our options, I agree with @carver 's suggestion to just remove the version dependency inside
ethportal-api
. There might be an option with configuring thePROJECT_ROOT
based on a flag, but that route introduces some management overhead that IMO is not worthwhile. This solution plays nicely with docker and all our current config (afaik)... and I'm not sure it's a huge loss to remove theVERSION
info from the cli. The only side affect that I can tell is that runningwill just return the string I replaced the version with, and if somebody needs to know the version, running the client quickly seems like a reasonable compromise.
To-Do