Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release 0.6.0 #218

Merged
merged 3 commits into from
Feb 5, 2024
Merged

Release 0.6.0 #218

merged 3 commits into from
Feb 5, 2024

Conversation

Restioson
Copy link
Owner

No description provided.

@Restioson Restioson changed the title Update xtra-macros cargo.toml Release 0.6.0 Feb 2, 2024
@Restioson
Copy link
Owner Author

@thomaseizinger: I've released from this branch now - can we merge these in? minimal set of changes to get it to work

@@ -1,7 +1,11 @@
[package]
name = "xtra-macros"
version = "0.1.0"
version = "0.6.0"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why does this need to track the version of the root crate?

Copy link
Owner Author

@Restioson Restioson Feb 2, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Arguably it does not, but I think this is a standard set in other crates - serde and serde_derive seem pegged, as do bevy and bevy_internal/bevy_dylib (and even bevy_input, which is not a dependency of bevy but a core extension crate). askama and askama_derive track eachother in the minor and major version. I would argue it's a bit simpler if we don't have, say xtra 0.8 depending on 0.2 of xtra_macros. At the very least, since we re-export xtra_macros from xtra, any breaking change (while in 0.x) requires bumping xtra alongside xtra_macros. Vice versa is not strictly true but is more likely as the API surface is so small now.

Apologies for doing this without consulting you. I did it without thinking too deeply about it at the time as I'd thought this was just kind of the standard practice. I'll make sure it doesn't happen again

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess it does makes things easier while we are in 0.x land and after 1.x, the difference doesn't matter too much :)

@thomaseizinger
Copy link
Collaborator

@thomaseizinger: I've released from this branch now - can we merge these in? minimal set of changes to get it to work

For the future, I'd suggest to not release from branches but only after they are merged. With squash merging, the commits in the branch never land on master, meaning you can't have a tag now that directly corresponds to the released code.

@Restioson
Copy link
Owner Author

Restioson commented Feb 2, 2024

Are you okay to just rebase merge this in to master then? Thanks for the tip, I didn't anticipate that.

@thomaseizinger
Copy link
Collaborator

Are you okay to just rebase merge this in to master then? Thanks for the tip, I didn't anticipate that.

Yes I think this would be a good idea! :)

@Restioson Restioson merged commit 0e150b6 into master Feb 5, 2024
6 checks passed
@Restioson Restioson deleted the release-0.6.0 branch February 5, 2024 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants