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

WIP 211 adjust tests after introduction of [.qenv in teal.code #1396

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

m7pr
Copy link
Contributor

@m7pr m7pr commented Oct 25, 2024

Companion of insightsengineering/teal.code#216
Result of this line https://github.com/insightsengineering/teal.code/pull/216/files#diff-66a61facc4f5ca86215d63af40e2645b364950439051784916316ef78d5073c8R69

teal.code::get_code_dependency now returns comments as well, so that when it is used in [. it creates an object that does contain comments in the final @code.

We might edit teal.code::get_code so it drops comments, so this will not be shown in the final extracted code.

@m7pr m7pr added the core label Oct 25, 2024
@m7pr m7pr changed the title WIP 211 adjust tests after introduction of subset in teal.code WIP 211 adjust tests after introduction of [.qenv in teal.code Oct 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant