You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I want to specify tagSorter and operationSorter in a Docker container. But according to the docs here, unfortunately it doesn't support passing tagSorter and operationSorter via environment variable.
Any reasons why these two options are unavailable on Docker? I think we could pass a value as string like 'alpha' / 'method' at least.
Describe the solution you'd like
What I did to solve this is to add tagSorter and operationSorter to the variable list defined in ./docker/configurator/variables.js like this:
I tested it and It worked well on my local environment, so if it looks good I'll make a PR.
Describe alternatives you've considered
Maybe we could sort at json file beforehand rather than passing sort options to Docker. But it would be still worth if we can pass the options to Docker.
Additional context
The text was updated successfully, but these errors were encountered:
This PR is still open. This is a nice addition to existing configurations, which are not available in docker version.
Could anyone review this PR and help to merge this to master?
Content & configuration
I want to specify
tagSorter
andoperationSorter
in a Docker container. But according to the docs here, unfortunately it doesn't support passingtagSorter
andoperationSorter
via environment variable.Any reasons why these two options are unavailable on Docker? I think we could pass a value as string like 'alpha' / 'method' at least.
Describe the solution you'd like
What I did to solve this is to add
tagSorter
andoperationSorter
to the variable list defined in./docker/configurator/variables.js
like this:I tested it and It worked well on my local environment, so if it looks good I'll make a PR.
Describe alternatives you've considered
Maybe we could sort at json file beforehand rather than passing sort options to Docker. But it would be still worth if we can pass the options to Docker.
Additional context
The text was updated successfully, but these errors were encountered: