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

Strip unused paths from the settings #167

Open
andyneff opened this issue Oct 6, 2023 · 0 comments
Open

Strip unused paths from the settings #167

andyneff opened this issue Oct 6, 2023 · 0 comments

Comments

@andyneff
Copy link
Member

andyneff commented Oct 6, 2023

When terra starts a runner, not all paths in the settings will be translates for a particular service because they are not always used. There is room for a process to strip unused paths.

Pros:

  • Less confusion for users who see a mixture of translated and untranslated paths
  • By removing the ability to use a path that is not translated, it prevents them from using it incorrectly when the paths should not be used in that service (without adding it to the service definition)

Cons:

  • When a file is written in the service runner, if it contains absolute paths, it would need patching. If someone was using an untranslated setting to accomplish this, it would break. But they really shouldn't be doing it this way.
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