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.

Sign up for GitHub

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

Jump to bottom

Remove __locals now that all tests access variables directly #54813

Closed
ilenia-magoni opened this issue May 15, 2024 · 0 comments · Fixed by #54828
Closed

Remove __locals now that all tests access variables directly #54813

ilenia-magoni opened this issue May 15, 2024 · 0 comments · Fixed by #54828
Labels
new python course scope: curriculum Lessons, Challenges, Projects and other Curricular Content in curriculum directory. status: PR in works Work in Progress (WIP) Issues.

Comments

@ilenia-magoni
Copy link
Contributor

freeCodeCamp/tools/client-plugins/browser-scripts/python-test-evaluator.ts

Lines 175 to 176 in 98e5ac1

// TODO: remove the next line, creating __locals, once all the tests access
// variables directly.

@ilenia-magoni ilenia-magoni added help wanted Open for all. You do not need permission to work on these. scope: curriculum Lessons, Challenges, Projects and other Curricular Content in curriculum directory. new python course labels May 15, 2024
@AndrewYturaldi AndrewYturaldi mentioned this issue May 16, 2024
Merged
4 tasks
@ilenia-magoni ilenia-magoni added status: PR in works Work in Progress (WIP) Issues. and removed help wanted Open for all. You do not need permission to work on these. labels May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
new python course scope: curriculum Lessons, Challenges, Projects and other Curricular Content in curriculum directory. status: PR in works Work in Progress (WIP) Issues.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(curriculum): removed line that creates __locals in python test ev… AndrewYturaldi/freeCodeCamp
1 participant
@ilenia-magoni