Taylor Steinberg
Taylor Steinberg
@techman83 - thanks for putting this together. I tried it out but ran into an error: https://github.com/posit-dev/posit-sdk-py/actions/runs/9353058789 It looks like the 'base' variable is undefined, which I believe stems from...
This is tricky. The underlying issue is that credentials are serialized into JSON format before being written to disk. The DER certificates are binary, which cannot be safely converted into...
Base64 might work. I'll run some experiments to verify.
For future reference, here is the PyPA specification for the requirements.txt file: For reference: https://pip.pypa.io/en/stable/reference/requirements-file-format/#requirements-file-format
It appears the CI failures are due to missing documentation. Once those are added, I think this is ready for final review.
It might be worth reverting from `pyproject.toml` to setup.py to support this: https://github.com/posit-dev/rsconnect-python/pull/668
> > It might be worth reverting from `pyproject.toml` to setup.py to support this: #668 > > [PyPA docs suggest pyproject.toml is the way](https://packaging.python.org/en/latest/flow/#the-configuration-file). Ya, it's just unconventional for the...
> I apologize in advance for the driveby comment about naming, but: given that "Package Manager" is one of Posit's products, I worry that calling this argument `--package-manager` is ambiguous...
I agree that making requirements.txt generation explicit instead of implicit is the correct direction. `pipreqs` looks like a suitable replacement for `pip freeze`. Our workflow implicitly assumes that the virtual...
Hi @tuxmaster5000, thanks for the pull request. Can you say a bit more about what scenario you are fixing with this change? I do not see any errors in my...