Грусть. Если организую MR, чтобы в функцию env вторым необязательным

Question

werf (чат)

Alexandr Dmitryuk

Грусть. Если организую MR, чтобы в функцию env вторым необязательным

параметром можно было передать значение по умолчанию, есть вероятность что его примут?

#devops #russian #werf

0

20.10.2023

5 ответов

8 просмотров

Alexandr Dmitryuk Автор вопроса

Ivan Mikheykin
Это не на уровне env вычисляется, это другой «слой...

Разве это не оно? https://github.com/werf/werf/blob/main/pkg/config/parser.go#L296

0

20.10.2023

Ilya Lesikov L.

Это не сломает ли существующее использование env? Не уверен, что можно сделать так, чтобы оба вариант работали: {{ env "MYENV" }} {{ env "MYENV" "default" }} Вероятнее всего можно сделать так, чтобы либо первое работало, либо второе. Давайте мы обсудим в команде, может перестанем возвращать ошибку из env вовсе. Тут, скорее всего, только два варианта — перестать возвращать ошибку или сделать новую функцию (envdef?)

0

20.10.2023

Alexandr Dmitryuk Автор вопроса

Ilya Lesikov L.
Это не сломает ли существующее использование env? ...

Параметр же опциональный и по дефолту будет не задан

0

20.10.2023

Ilya Lesikov L.

Alexandr Dmitryuk
Параметр же опциональный и по дефолту будет не зад...

кстати действительно похоже вот так можно опциональные аргументы задать https://go.dev/play/p/6x2X8yZABnG

0

20.10.2023