alexshikov / bitrise-step-appcenter-dsym-upload

A Bitrise step to upload dSYM to Microsoft AppCenter
MIT License
1 stars 1 forks source link

Environment variable being overwritten #4

Closed Kymer closed 10 months ago

Kymer commented 3 years ago

I had an issue where reading the $BITRISE_DSYM_PATH environment variable after this step ran in my workflow it returned an empty string. I noticed the following line envman add --key BITRISE_DSYM_PATH. It seems like that overwrites the value, no? Is this somehow intentional?

I'd be happy to create a PR, but first wanted to check I'm not going crazy and might be misunderstanding something, somehow hah.

alexshikov commented 3 years ago

Hi @Kymer that's a good question. I don't remember the reason of adding this line. It's either a copy paste from another example, or this line was required to make the script work :)

If you have a fix on your mind and you could test it, I would be glad to accept the PR.