insta icon indicating copy to clipboard operation
insta copied to clipboard

Make insta CLI work outside of cargo, and distribute it via npm

Open untitaker opened this issue 1 year ago • 2 comments

sentry uses insta to review snapshot files generated by python code. There is an ad-hoc implementation of insta-the-library vendored somewhere in sentry.utils.pytest.fixtures.

People who want to review those snapshots currently have to install insta via cargo. What if insta used cargo-dist to distribute insta CLI on npm instead? Then sentry can add it to the developer's setup using yarn add --dev. Yes, as a frontend dev dependency, but everybody knows how to install those. Better would be pypi package, but who has time for that.

untitaker avatar Jun 25 '23 23:06 untitaker

Another option would be to make it self installable like rye does from github releases maybe?

mitsuhiko avatar Jun 26 '23 13:06 mitsuhiko

fwiw i tried cargo-dist on another project and it isn't really hard to get it working for both npm + prebuilt binaries, can recommend

untitaker avatar Jun 26 '23 14:06 untitaker

cargo-insta now has binaries. Closing this as resolved.

mitsuhiko avatar Mar 28 '24 11:03 mitsuhiko