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

Annotate .gitmodules with branch and tag #693

Merged
merged 4 commits into from
Feb 21, 2024

Conversation

creationix
Copy link
Member

@creationix creationix commented Feb 21, 2024

I'm working on an experiment to make submodules easier to work with. Part of that is annotating .gitmodules with information to make it so tools can automatically update dependencies.

The branch field is actually supported already by upstream git itself.

The tag field is my invention and my tool may be using it to do semver updates based on the tag name and available upstream tags.

I'm not convinced the tag needs to be documented here since the commit hash is already in git and it possible to map that back to tags. I've removed tag from the PR for now till I'm convinced it needs to be in the file.

@creationix creationix marked this pull request as draft February 21, 2024 12:56
@creationix creationix marked this pull request as ready for review February 21, 2024 13:04
@creationix creationix enabled auto-merge (squash) February 21, 2024 13:08
@SinisterRectus
Copy link
Member

Hi Tim! My 👍 just meant I approve of anything that makes the build process easier, but I don't know enough to properly sign off on this.

@creationix creationix enabled auto-merge (squash) February 21, 2024 13:09
@creationix
Copy link
Member Author

No worries, thanks for the 👍🏽

@creationix creationix requested review from zhaozg and squeek502 and removed request for SinisterRectus February 21, 2024 13:11
@creationix creationix merged commit 372da9d into master Feb 21, 2024
26 checks passed
@Bilal2453
Copy link
Contributor

This sounds interesting. Depending on how it goes we might eventually make use of this in Lit as well!

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.

4 participants