Commit 9c6eec2

committed

fix(bump): fixed environment variables in bump hooks

During development of commitizen-tools#644, merging the hook environment variables with existing OS variables broke. This commit fixes that.

1 parent 1ff575d commit 9c6eec2Copy full SHA for 9c6eec2

File tree

-1

lines changed

-1

lines changed

Lines changed: 3 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,8 @@`
`3`	`3`	`from commitizen import cmd, out`
`4`	`4`	`from commitizen.exceptions import RunHookError`
`5`	`5`
	`6`	`+import os`
	`7`	`+`
`6`	`8`
`7`	`9`	`def run(hooks, _env_prefix="CZ_", **env):`
`8`	`10`	`if isinstance(hooks, str):`
`@@ -25,7 +27,7 @@ def run(hooks, _env_prefix="CZ_", **env):`
`25`	`27`	`def _format_env(prefix: str, env: dict[str, str]) -> dict[str, str]:`
`26`	`28`	`"""_format_env() prefixes all given environment variables with the given`
`27`	`29`	`prefix so it can be passed directly to cmd.run()."""`
`28`		`- penv = dict()`
	`30`	`+ penv = os.environ`
`29`	`31`	`for name, value in env.items():`
`30`	`32`	`name = prefix + name.upper()`
`31`	`33`	`value = str(value) if value is not None else ""`

Comments

(0)