search

projectfluent / python-fluent

Python implementation of Project Fluent
https://projectfluent.org/python-fluent/
Other
210 stars 26 forks source link

Copy fluent.runtime/runtests.py to fluent.syntax #155

Closed stasm closed 1 year ago

stasm commented 4 years ago

VS Code doesn't like the fact that fluent.syntax/runtests.py looks like a Bash script (it's a symlink) rather than a Python script as the extension would suggest.

stasm commented 4 years ago

I think that would be good, but in this PR I'd like to just unblock myself from being able to use the Python interpreter in VS Code. Two copies is still OK, so I'd suggest to factor the script out the root dir once we have maybe 3 or 4.

stasm commented 4 years ago

My quick glance says that moving the file wouldn't require changes to runtests itself, just the callsites would need to change.

I tried that and it did work when I invoked ../runtests.py manually from the CLI. Unfortunately, tox was no longer able to find it and I didn't investigate further.

eemeli commented 1 year ago

Closing, as modern VS Code correctly detects the symlinked file as Python.