cargo-unleash icon indicating copy to clipboard operation
cargo-unleash copied to clipboard

deploy - use tag as version

Open drahnr opened this issue 3 years ago • 5 comments

Closes #67

drahnr avatar Feb 02 '22 17:02 drahnr

But this doesn't update the Cargo.toml that is comitted. Only temporary. Shouldn't we just bump it manually and then release?

athei avatar Feb 04 '22 11:02 athei

Well, we could inverse the relation. Make the manifest the truth and create a git tag after release (like cargo release does) - downside: The gh actions requires push access to master (not sure if there is a delta between tags and commits).

drahnr avatar Feb 04 '22 11:02 drahnr

This sounds much better to me.

(not sure if there is a delta between tags and commits).

Don't understand what you mean. It would make the tag to a fixed commit hash and also release this very ref. If someone pushes to master in-between this wouldn't matter. Was this your concern?

athei avatar Feb 04 '22 12:02 athei

Don't understand what you mean. It would make the tag to a fixed commit hash and also release this very ref. If someone pushes to master in-between this wouldn't matter. Was this your concern?

Access rights for the github action to tag (aka write to the repo would be required) - that's usually not great since the action can be modified by contributors, hence not great.

drahnr avatar Feb 04 '22 12:02 drahnr

Aren't changes to github actions only executed once they are merged to master for this reason as a security measure (or at least you can enable that)?

athei avatar Feb 04 '22 12:02 athei