Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Better handling of set_env #1784

Merged
merged 5 commits into from Jan 11, 2021
Merged

Better handling of set_env #1784

merged 5 commits into from Jan 11, 2021

Conversation

gaborbernat
Copy link
Member

@gaborbernat gaborbernat commented Jan 10, 2021

Resolves #1779.
Resolves #1776.

Also introduces a lot of other fixes for things without a ticket and discovered while fixing these.

@codecov
Copy link

codecov bot commented Jan 10, 2021

Codecov Report

Merging #1784 (8acd7b7) into rewrite (096f6a8) will increase coverage by 0.02%.
The diff coverage is 99.65%.

Impacted file tree graph

@@             Coverage Diff             @@
##           rewrite    #1784      +/-   ##
===========================================
+ Coverage    99.73%   99.75%   +0.02%     
===========================================
  Files          134      137       +3     
  Lines         6775     6982     +207     
  Branches       646      678      +32     
===========================================
+ Hits          6757     6965     +208     
+ Misses           7        6       -1     
  Partials        11       11              
Flag Coverage Δ
tests 99.75% <99.65%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/tox/execute/local_sub_process/__init__.py 95.48% <ø> (ø)
tests/session/cmd/test_show_config.py 100.00% <ø> (ø)
src/tox/config/loader/str_convert.py 98.52% <83.33%> (+1.34%) ⬆️
src/tox/config/cli/ini.py 100.00% <100.00%> (ø)
src/tox/config/loader/api.py 100.00% <100.00%> (ø)
src/tox/config/loader/ini/__init__.py 100.00% <100.00%> (ø)
src/tox/config/loader/ini/factor.py 100.00% <100.00%> (ø)
src/tox/config/loader/ini/replace.py 100.00% <100.00%> (ø)
src/tox/config/loader/stringify.py 100.00% <100.00%> (ø)
src/tox/config/main.py 100.00% <100.00%> (ø)
... and 24 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 096f6a8...8acd7b7. Read the comment docs.

@gaborbernat
Copy link
Member Author

Blocked on pypa/virtualenv#2046

@gaborbernat gaborbernat changed the title Fix CI Better handling of set_env Jan 10, 2021
- support set env values in env replacer
- support using set env key-values within set env
- if a set env refers to itself then the value will be the os.environ
content if present (resolve circular dependency)
- raise error if resolving replacements cause a circle

Signed-off-by: Bernát Gábor <bgabor8@bloomberg.net>
Signed-off-by: Bernát Gábor <bgabor8@bloomberg.net>
Signed-off-by: Bernát Gábor <bgabor8@bloomberg.net>
Signed-off-by: Bernát Gábor <bgabor8@bloomberg.net>
Signed-off-by: Bernát Gábor <bgabor8@bloomberg.net>
@gaborbernat
Copy link
Member Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant