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

Version numbers do not match #15

Closed
t-richards opened this issue Jun 25, 2018 · 4 comments
Closed

Version numbers do not match #15

t-richards opened this issue Jun 25, 2018 · 4 comments
Assignees

Comments

@t-richards
Copy link

t-richards commented Jun 25, 2018

Version information

  • Crystal: 0.25.0
  • migrate.cr: 0.2.2

Problem summary

The version numbers contained within code/metadata do not match the git tag. This causes funny behavior in shards, For example, running $ shards check in a project which depends on migrate.cr will always show that the dependencies are not satisfied, which will cause it to re-fetch all dependencies (this is slow and annoying, especially with lots of deps)

Mismatched version locations

@vladfaust
Copy link
Owner

Wow, never thought of that. So, each small commit would require to bump versions in those two files? That's mess.

@t-richards
Copy link
Author

each small commit would require to bump versions in those two files?

Not necessarily each commit. But each tagged release, yes.

@vladfaust
Copy link
Owner

There is no need in duplication of a tag within the code itself. It's just a waste of time (waste bigger than re-fetching). Version locking works perfectly with tags-only approach - shards installed and udpated correctly and the repository isn't bloated with useless bumps.

Your issue looks more like shards issue itself. I encourage you to create an issue there to discuss this.

Or you can try to convince me more 😃

@t-richards
Copy link
Author

You're right, updating Migrate::VERSION in the code isn't strictly necessary, but nice to have. I don't have a good argument for why shards needs the version in shards.yml, since it fetches version information from the tags anyway, but this seems to be how it works today.

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

No branches or pull requests

2 participants