Skip to content
New issue

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

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

Already on GitHub? # to your account

Add versions to dist artifacts #288

Closed
wants to merge 3 commits into from
Closed

Add versions to dist artifacts #288

wants to merge 3 commits into from

Conversation

wheresvic
Copy link

@wheresvic wheresvic commented Sep 21, 2017

This PR is to have the dist artifacts versioned to manage upgrades. Closes #289

@wheresvic
Copy link
Author

@picturepan2 Any comment on this MR?

@picturepan2
Copy link
Owner

@victorparmar Why do you think this change is necessary?

@wheresvic
Copy link
Author

wheresvic commented Sep 30, 2017

@picturepan2 As spectre matures, it would be good to offer built in versioning for users to manage updates. I have been using spectre.css in a few projects since 0.3.x and it is getting difficult for me to see which project is using which version.

Of course I can always postfix the version myself but that is an extra step for me to remember and do. Other mature projects such as jQuery also come versioned by default!

@wheresvic
Copy link
Author

wheresvic commented Oct 2, 2017

@picturepan2 I'm sorry to be a pain in the ass but I have a few projects that are waiting to be updated and it would be great to have versioning built in. Otherwise I will need to make a custom automated solution to generate my own builds.

It would be great if you could let me know if you consider adding this feature or don't plan on it at all.

Thanks,

@Geczy
Copy link

Geczy commented Oct 4, 2017

Bootstrap versions by adding commented code to the top of their dist. They don't version their filenames

/*!
 * Bootstrap v4.0.0-beta (https://getbootstrap.com)
 * Copyright 2011-2017 The Bootstrap Authors
 * Copyright 2011-2017 Twitter, Inc.
 * Licensed under MIT (https://github.com/twbs/bootstrap/blob/master/LICENSE)

Would this work?

@picturepan2
Copy link
Owner

picturepan2 commented Oct 4, 2017 via email

picturepan2 added a commit that referenced this pull request Oct 5, 2017
@picturepan2 picturepan2 mentioned this pull request Oct 10, 2017
@picturepan2
Copy link
Owner

@Geczy Version number is added to CSS files in v0.4.4.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants