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

Fix broken Bower-Chosen process #2527

Closed
wants to merge 1 commit into from
Closed

Conversation

pfiller
Copy link
Contributor

@pfiller pfiller commented Feb 8, 2016

@harvesthq/chosen-developers

In #2449, the line that grabs our latest tag was changed to git tag --sort=v:refname | tail -1. Unfortunately, that line errors out on Travis.

See?

Fortunately, we don't really need it. git tag -l lists all tags in alphabetical order. As long as we're
consistent (that's the key) with our tagging, the last alphabetical version will also be the most recent version. That should be true going forward.

chosen patrickfiller$ git tag -l
0.12.1
0.13.0
0.14.0
0.9
1.0.0
1.4.0
1.4.2
semver
v0.10.0
v0.10.1
v0.11.1
v0.12.0
v0.9
v0.9.1
v0.9.10
v0.9.11
v0.9.12
v0.9.13
v0.9.14
v0.9.15
v0.9.2
v0.9.3
v0.9.4
v0.9.5
v0.9.6
v0.9.7
v0.9.8
v0.9.9
v1.1.0
v1.2.0
v1.3.0
v1.4.0
v1.4.1
v1.5.0

Simple change. What say you?

`git tag -l` lists all tags in alphabetical order. As long as we're
consistent* (that's the key) with our tagging, the last alphabetical
version will also be the most recent version.
@pfiller pfiller mentioned this pull request Feb 8, 2016
@koenpunt
Copy link
Collaborator

koenpunt commented Feb 8, 2016

we could also update git in the before task?

@koenpunt
Copy link
Collaborator

koenpunt commented Feb 8, 2016

And leave the sorting as-is

@koenpunt
Copy link
Collaborator

koenpunt commented Feb 8, 2016

Because that's the problem here right? That the version of git on travis is not the latest?

@koenpunt
Copy link
Collaborator

koenpunt commented Feb 8, 2016

I now remember why I didn't use git tag -l; it ranks 0.9.1 higher than 0.12.0

image

@koenpunt koenpunt closed this Feb 18, 2016
@tjschuck tjschuck deleted the update-git-tag-grabbing branch February 18, 2016 20:36
@pfiller
Copy link
Contributor Author

pfiller commented Feb 18, 2016

@koenpunt's #2528 fixed the underlying issue so we can move on until the whole thing breaks again in a few months...

@koenpunt
Copy link
Collaborator

until the whole thing breaks again in a few months...

I think I don't know what you mean with this? What are you referring to?

@tjschuck
Copy link
Member

@pfiller making an update in a few months:

@pfiller
Copy link
Contributor Author

pfiller commented Feb 18, 2016

I think I don't know what you mean with this? What are you referring to?

Nothing specific. Seems like we give an unreasonable proportion of this project's time to making things work for the Bower crowd. I'm mostly just being a krabby patty.

@koenpunt
Copy link
Collaborator

So what now, we're prepping for a 2.0? With @mlettini's css-rework et al?

@pfiller
Copy link
Contributor Author

pfiller commented Feb 20, 2016

I've spent a good amount of time reading through open issues and thinking about a philosophy for Chosen. We haven't had anything like that to this point and I think it's fair to say the project has grown quite stale as a result. I hate looking up and seeing 79 open PRs.

I'm on vacation for the next two weeks, but I think it'd be nice to chat about a direction for Chosen when I get back. Would you be up for a Hangout then?

@koenpunt
Copy link
Collaborator

Sounds like a plan! Just let me know when you're back.

# 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