search

lewagon / nbresult

Testing Library for Jupyter Notebooks
https://pypi.org/project/nbresult/
MIT License
8 stars 2 forks source link

Replace hardcoded `python3` with `sys.executable` #12

Closed ssaunier closed 3 months ago

ssaunier commented 3 months ago

Fixes https://github.com/lewagon/help/issues/5229

Re-reading the long conversation happenning, I came to the conclusion that it does not harm to have the full path of the current running Python instead of hardcoded (kudos @gmanchon for finding this one).

I suggest we merge this one and re-release the nbresult package to 0.1.0, then we do a new release of https://github.com/lewagon/data-runner

WDYT?

krokrob commented 3 months ago

@ssaunier do you have an opened PR for the data-runner?