Skip to content

Comments

Escaping nodeids in query contexts#51

Merged
jmafoster1 merged 4 commits intomainfrom
jmafoster1/50-escape-parameterised-nodeids
Feb 19, 2026
Merged

Escaping nodeids in query contexts#51
jmafoster1 merged 4 commits intomainfrom
jmafoster1/50-escape-parameterised-nodeids

Conversation

@jmafoster1
Copy link
Contributor

Closes #50. This allows for parameterised tests and (hopefully) other special characters.

@github-actions
Copy link

github-actions bot commented Feb 16, 2026

MegaLinter analysis: Success

Descriptor Linter Files Fixed Errors Warnings Elapsed time
✅ PYTHON black 21 0 0 0 1.06s
✅ PYTHON pylint 21 0 0 5.64s

See detailed reports in MegaLinter artifacts

Your project could benefit from a custom flavor, which would allow you to run only the linters you need, and thus improve runtime performances. (Skip this info by defining FLAVOR_SUGGESTIONS: false)

  • Documentation: Custom Flavors
  • Command: npx mega-linter-runner@9.3.0 --custom-flavor-setup --custom-flavor-linters PYTHON_PYLINT,PYTHON_BLACK

MegaLinter is graciously provided by OX Security

@codecov
Copy link

codecov bot commented Feb 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@jmafoster1 jmafoster1 force-pushed the jmafoster1/50-escape-parameterised-nodeids branch from 5fbccf3 to e960ec1 Compare February 16, 2026 14:43
@jmafoster1 jmafoster1 marked this pull request as ready for review February 16, 2026 14:52
@gkapfham
Copy link
Collaborator

@jmafoster1 have you tried this with the example project that I mention in issue #50 ?

@gkapfham
Copy link
Collaborator

I will try to test this PR as soon as I have time, perhaps this week. Thanks @jmafoster1.

@jmafoster1
Copy link
Contributor Author

@jmafoster1 have you tried this with the example project that I mention in issue #50 ?

Yes, this is how I worked out what the problem was. It runs fine on that project now

@jmafoster1 jmafoster1 merged commit 800dad8 into main Feb 19, 2026
30 checks passed
@jmafoster1 jmafoster1 deleted the jmafoster1/50-escape-parameterised-nodeids branch February 19, 2026 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Installing Plugin from PyPI Causes a Crash When Using Default Deflaker While Running Test Suite

2 participants