Félix Saparelli

Results 581 comments of Félix Saparelli

Can you explain the bug a little more? I see three successful runs here

I see. Is this a problem with anything else than CMD? I'm tempted to special-case CMD to translate UNC paths for it specifically, but leave UNCs for other things (like...

I don't have any time to work on Watchexec at the moment, but would accept a PR to revert that change / disable UNC for everyone.

Registering interest in this implementation as we're going to add support for checking sigstore in binstall in the near-but-not-defined future, so would be annoying if we were incompatible by accident

FTR The design we're thinking of with binstall is to use the cosign (sign-blob and) verify-blob workflows (but in Rust, not through the CLI go tools) directly on the packages...

Going to note that github has released this https://github.blog/2024-05-02-introducing-artifact-attestations-now-in-public-beta/ which might simplify the story at least for github-actions.

Noting that the workarounds also cause the conditional to not syntax-highlight anymore (a more minor annoyance to be sure, though).

Another workaround which does preserve syntax highlighting is to wrap with `()` instead: ```diff - - if: "!startsWith(github.ref, 'refs/heads/release/')" + - if: (!startsWith(github.ref, 'refs/heads/release/')) ```

Consider using the notify crate directly for that kind of integration.