[Feature Request]: internal `tenv` variable might cause name clash

insightsengineering / tlg-catalog

A catalog of Tables, Listings and Graphs (TLGs) created with NEST R packages

https://insightsengineering.github.io/tlg-catalog/

Other

18 stars 3 forks source link

[Feature Request]: internal `tenv` variable might cause name clash #214

Open pawelru opened 4 months ago

pawelru commented 4 months ago

Feature description

Rename tenv into something less likely to be used in articles, e.g. .tenv or .__tenv__.

Code of Conduct

[X] I agree to follow this project's Code of Conduct.

Contribution Guidelines

[X] I agree to follow this project's Contribution Guidelines.

Security Policy

[X] I agree to follow this project's Security Policy.

shajoezhu commented 4 months ago

hi @pawelru , I was wondering where was clash happened? thanks

pawelru commented 4 months ago

The clash is possible when one would use tenv in any of the code chunk in the article. Luckily that's not the case for now so this is not urgent. I noticed this when working on the readme documentation where I explained this a little.