passenv statement should accept multi-line list #259
Comments
Original comment by @hpk42 add changelog to fix issue259: passenv is now a line-list which allows to intersperse → <> |
Original comment by @stefano-m @hpk42, I have updated the PR following your advice. I hope I got everything right. |
Original comment by @hpk42 Thanks for the PR and the discussion. I think allowing multi-line-list and splitting each non-comment line by hand would be workable. We need to check/test then that it works with factor-syntax (but i think it should). It should be backward compatible unless i am mistaken. @stefano-m could you try this multi-line-list approach? |
Original comment by @stefano-m I have created pull request #166 that attempts to address this issue. |
Original comment by @stefano-m I second this request. I am afraid that this would break backwards compatibility though, and the I have been (unsuccessfully) trying to modify the tox tests and code so that both styles work, i.e.
and
but since the configuration specifies that the passenv option should be a Interestingly, if I change the passenv parameter to be of type An advantage that I see in having a multi-line list is the ability to add comments. For example:
Such comments document why the variables need to be passed and help to maintain the If someone has any pointers, I'll try and make this work. |
It would be useful to insert a large list of environment variables to 'passenv' instead of inserting a single long line.
The text was updated successfully, but these errors were encountered: