-
Notifications
You must be signed in to change notification settings - Fork 41
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
Two proposed icons #3
Comments
@leeper apologies - somehow I managed to not be watching my own repo, and missed this issue. A LONG time ago. These are both good ideas, though yeah, I'm not quite sure how to handle the hyperlinking. My gut reaction is to keep the hyperlink to the standards doc and have the new location somewhere in the README itself, but that does defeat the machine-readable metadata... I'm certainly open to ideas. |
For the record, I really like this idea, and it's deeply bothering me that I can't seem to figure out a "nice" way to implement it (i.e. keep references to both the standards doc and the new location). If anyone had any suggestions I'm certainly open to them. |
Just revisiting this, and commenting for the record: I like the idea, but I don't feel that I can make the decision on the hyperlink target myself, as I'm not planning on using this status personally. If anyone else has input, I'd be very interested to hear it. Until then, I'm just leaving this open to see if it gets any more interest. |
My opinion Just leave it simple, But my level of expertise is none, I like the badges as they are, used them on 2 projects. |
@RMHogervorst thanks for chiming in. All opinions are welcome and valued... I haven't done much with this project in a while as it fits my needs in its current state, and I realize I've let this issue sit for an embarrassingly long amount of time. I don't think I can really give this a lot of time right now, but I'll try to get back to it. I suppose that for the markup formats that support alt text, the moved to URL could be encoded there... |
Thanks for the reply @jantman , If someone wants to add or remove things they can always fork your project. Anyway, |
Sorry for the delayed response. I'd really like this to become somewhat of a standard for people who use it, so I'd prefer to incorporate any desired changes here, rather than having people run off of a fork - that works for changing badges and text, but not so much for the specification which is intended to be machine-parsable. Automated deployment is certainly an option if you want to go that way, provided that the repo status text ends up in the repo's README. I intentionally stayed away from that, since (1) you can always pull last commit information from the repo itself, and (2) I wanted this to be a way of communicating the human maintainer's intent - i.e. I have plenty of projects that might not have had a commit in years but I still consider active and supported, and some that had commits a week or two ago but I've decided to abandon. -Jason |
I now use the text from your website for creation of badges. my implementation, just creating a link based on status, is here: I think that keeping it to one standard is indeed better. It would be bad if things break later on. In answer to keeping the text relatively the same and adding a link, maybe a relative link?
and then a seperate file with the link? It's still a bit of a workaround though. |
So... @RMHogervorst I'm not sure how that would render across different markups. Alternate theories:
|
I hadn't thought about other markups. I guess your theories work just as well. |
It's been 9 months since any further commentary. Any further thoughts on whether this is needed, or implementation? |
Maybe the simplest thing is just add a colon so |
Would that really even require updating the badges, or do you think it would suffice to recommend that users add text so it appears like: While it would complicate the parsing a bit, the specification could be updated as follows:
Thoughts, @leeper and @RMHogervorst ? |
Works for me. |
Ok, cool. I did a quick GitHub search while looking into #19 and found that, excluding my own repos, there are over 1,200 code hits for repostatus.org badge URLs. I think it's time for me to decide that decisions about adding statuses should get community feedback... Do you think that, from a usability/user experience point of view, there's really sufficient difference between "Moved" and "Forked" for them to be separate statuses? |
@HoverBaum @RMHogervorst @roryokane @sagikazarmark @jennybc do any of you have thoughts on either this issue or #21 ? |
Following the discussion in #19 and realizing that repostatus wants to mainly talk about two things, namely:
I have problems sorting "moved" into this. A repository that was moved would from my point of view be "Unsupported" if the code reached a stable state or be "Concept" if the code never reached a stable state but illustrates a direction that could be taken. That is assuming how I interpret "Concept" (see #22 ). Now at the same time I see how the information that a project got moved to a different repository would be a very useful addition to "Unsupported" as it would inform a user that they can find support elsewhere. However I feel that this would turn into repostatus being more of a "projectstatus" kind of thing. Walking down this road might open us up to having to include more and more statuses that in a similar way talk more about the status of a project rather than a single repository. If the status "Moved" should be included I feel that @jantman suggestion with a badge followed by a link would work nicely. Will look at 21 after the weekend. |
I agree with @HoverBaum that repo status should rather about the repo. I could imagine a separate badge for an officially accepted fork. Another (more common) case might be migrating a repo to an organization from a personal account. In that case a repo should be discontinued, and moved to a new repo. So this need is valid, but I would separate the use case itself from repostatus. As said a discontinued badge (can't recall the exact name here) and a separate link or moved to badge would work for me. |
Works for me too. I dont mind adding an extra badge with discontinued, or just one badge. |
Thanks to all for the input. Since it's the weekend I'm going to let this sit for another day or two in case anyone else wants to offer input, but I'll come back to it this coming week. |
Apologies for letting this sit so long, I've had some personal issues that prevented me from getting much work done on this. I've added a "Moved" badge in the issues/3 branch.
Thoughts? If this is good, I'll merge it and cut a release sometime this week when I also handle the other open issues. |
If interested, feel free to also comment on the PR, #24 |
The "Moved" status was added in 2.0.0, released just now. |
I think this could benefit from two additional statuses/icons:
In both cases, I could imagine hyperlinks being used to point to the new locations of the code, but that would go against the current standard of pointing back to the standards document.
The text was updated successfully, but these errors were encountered: