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

Deprecate master branch completely (remove it) #9628

Closed
EmilyGraceSeville7cf opened this issue Dec 15, 2022 · 24 comments
Closed

Deprecate master branch completely (remove it) #9628

EmilyGraceSeville7cf opened this issue Dec 15, 2022 · 24 comments
Assignees
Labels
clients Issues pertaining to a particular client or the clients as whole. decision A (possibly breaking) decision regarding tldr-pages content, structure, infrastructure, etc.

Comments

@EmilyGraceSeville7cf
Copy link
Contributor

EmilyGraceSeville7cf commented Dec 15, 2022

master branch must be deprecated in the future, we only have to decide when.

@EmilyGraceSeville7cf EmilyGraceSeville7cf self-assigned this Dec 15, 2022
@EmilyGraceSeville7cf EmilyGraceSeville7cf changed the title Deprecate master branch completely (remove it) in 3 mounths Deprecate master branch completely (remove it) in 3 months Dec 15, 2022
@kbdharun kbdharun added the decision A (possibly breaking) decision regarding tldr-pages content, structure, infrastructure, etc. label Dec 15, 2022
@kbdharun
Copy link
Member

kbdharun commented Dec 15, 2022

I would suggest starting a discussion regarding this in our Gitter/Matrix Chatroom.

@waldyrious waldyrious mentioned this issue Dec 20, 2022
5 tasks
@MasterOdin
Copy link
Collaborator

The transition to main happened around May 1, 2021 (#5868). I would propose that at this point master gets deprecated immediately to be removed on May 1, 2023, giving all clients a full 2 years since main was made available to make the transition. A follow-up would be to then go through the list of clients on the wiki, and for any that are still using master, create an issue alerting them that master will be removed, and for them to transition.

@kbdharun
Copy link
Member

Interestingly, Even our python client uses the master branch. I created this PR tldr-pages/tldr-python-client#200 to update it.

@kbdharun
Copy link
Member

kbdharun commented Dec 27, 2022

I noticed some clients use the master branch version of the zip URL from https://github.com/tldr-pages/tldr-pages.github.io too. Should we keep the branch over there or depreciate it along with this?

@pixelcmtd
Copy link
Member

Interestingly, Even our python client uses the master branch.

I think I'll have to do another round of checking every tldr client before we can even think about removing master.

@kbdharun
Copy link
Member

kbdharun commented Dec 27, 2022

Interestingly, Even our python client uses the master branch.

I think I'll have to do another round of checking every tldr client before we can even think about removing master.

Thanks, I am almost done (Just some clients to go), I will add the list here in an hour.

@kbdharun
Copy link
Member

kbdharun commented Dec 27, 2022

Clients to fix:

@kbdharun
Copy link
Member

Done you can check the above list @pixelcmtd

@kbdharun kbdharun added the clients Issues pertaining to a particular client or the clients as whole. label Dec 27, 2022
@pixelcmtd pixelcmtd changed the title Deprecate master branch completely (remove it) in 3 months Deprecate master branch completely (remove it) Dec 27, 2022
@pixelcmtd
Copy link
Member

Done you can check the above list

@kbdharun Just saw this now, gg I've wasted the last 2 or so hours of my life :c

@kbdharun
Copy link
Member

kbdharun commented Dec 27, 2022

Done you can check the above list

@kbdharun Just saw this now, gg I've wasted the last 2 or so hours of my life :c

Oh, 😅 🤣 . That's actually good too. If I accidentally missed something you could have checked.

kbdharun added a commit to kbdharun/py-tldr that referenced this issue Dec 27, 2022
[About 1.5 years ago](tldr-pages/tldr#5868), we deprecated the master branch in favor of the main branch. We intend to remove it [soon, likely by May 1st 2023](tldr-pages/tldr#9628).
kbdharun added a commit to kbdharun/tldr-sh-client that referenced this issue Dec 27, 2022
[About 1.5 years ago](tldr-pages/tldr#5868), we deprecated the master branch in favour of the main branch. We intend to remove it [soon, likely by May 1st 2023](tldr-pages/tldr#9628).
kbdharun added a commit to kbdharun/rust-tldr that referenced this issue Dec 27, 2022
[About 1.5 years ago](tldr-pages/tldr#5868), we deprecated the master branch in favour of the main branch. We intend to remove it [soon, likely by May 1st 2023](tldr-pages/tldr#9628).
@kovmir
Copy link

kovmir commented Dec 28, 2022

Thank you! I have merged your pull request.

ronan696 pushed a commit to ronan696/keypirinha-tldr that referenced this issue Feb 10, 2023
[About 1.5 years ago](tldr-pages/tldr#5868), we deprecated the master branch in favour of the main branch. We intend to remove it [soon, likely by May 1st 2023](tldr-pages/tldr#9628).
@kbdharun
Copy link
Member

kbdharun commented Apr 9, 2023

Update: Reminded, the repo owners of the unmerged PRs about the change, I will do it again after 15 days.

@pixelcmtd
Copy link
Member

It's 1 May 04:25 in UTC right now. Does anyone have further concerns or can we delete the master branch in a few hours?

@kbdharun
Copy link
Member

kbdharun commented May 1, 2023

It's 1 May 04:25 in UTC right now. Does anyone have further concerns or can we delete the master branch in a few hours?

Can we start a discussion regarding this in the chatroom, in order to get other's opinions too.

I think we could do it, and update wiki to reflect the remaining clients here as borked or outdated. Then since we updated client specification too then we can make a release.

@pixelcmtd
Copy link
Member

I think we could do it, and update wiki to reflect the remaining clients here as borked or outdated.

I've already spent my last few days thinking about whether to make a table of unmaintained, broken and outdated clients or just mark and move them to the ends of their respective tables.

Could also tie in well with the client spec compliance/certification plans I've been thinking about proposing for months (maybe even years, I've lost track) by now

@kbdharun
Copy link
Member

kbdharun commented May 1, 2023

I think we could do it, and update wiki to reflect the remaining clients here as borked or outdated.

I've already spent my last few days thinking about whether to make a table of unmaintained, broken and outdated clients or just mark and move them to the ends of their respective tables.

Yeah that would be the best

Could also tie in well with the client spec compliance/certification plans I've been thinking about proposing for months (maybe even years, I've lost track) by now

Agreed, we should mark if the community clients conform to the client specification too.

@SethFalco and @waldyrious what do you think?

@pixelcmtd
Copy link
Member

Oh god, do you really want to turn this into a discussion about that? I was very not ready for that yet, but whatever. So my ideas were basically something along these lines:

  • Figure out whether we can put the client spec into a machine-readable/algorithmic/? form
  • If yes, do that. Otherwise, make a checklist at the end of the spec with a lot of commands and what they're supposed to do
  • Optionally, make a tool that you can give the path to a tldr client and it tells you whether it's compliant (this could probably be a 10 line shell script)
  • Sort the client list by compliance
  • Spend like 10 minutes of design work making certificate "badges"
  • Allow clients that are 100% compliant with a version of the client spec to carry the respective badge (after they've been certified by someone from tldr; each client vendor can open an issue if they think they're compliant and if 2 tldr members/contributors sign off on it, they are listed on the wiki with that compliance and afterwards allowed to carry that badge when advertising their project etc)

@kbdharun
Copy link
Member

kbdharun commented May 1, 2023

Allow clients that are 100% compliant with a version of the client spec to carry the respective badge (after they've been certified by someone from tldr; each client vendor can open an issue if they think they're compliant and if 2 tldr members/contributors sign off on it, they are listed on the wiki with that compliance and afterwards allowed to carry that badge when advertising their project etc)

I have been thinking for a while, and it would make sense with this change. How about we limit Wiki editing to collaborators and add an issue template for client authors to request their Client addition or if it's already present then updation of details in the Wiki?

It will also prevent Spam like this one https://github.com/tldr-pages/tldr/wiki/Home/0aff51b61cbf32b281a60622766766ced569032c where somebody edited the NodeJS link, I will revert this now.

@pixelcmtd
Copy link
Member

How about we limit Wiki editing to collaborators

Why the f- isn't it right now? Seems kinda obvious, sure.

How about we [...] add an issue template for client authors to request their Client addition or if it's already present then updation of details in the Wiki?

Taking a more incremental approach, we should probably make such an issue template right now.

@SethFalco
Copy link
Member

SethFalco commented May 1, 2023

table of unmaintained, broken and outdated clients or just mark and move them to the ends of their respective tables.

Let's not move them to the end of a table.
They should be separated entirely, and out of sight for new users just looking for a client. It'd be better, imo to have the list of maintained clients that follow our client specifications, and then a separate table (in a details tag) of unmaintained clients.

Figure out whether we can put the client spec into a machine-readable/algorithmic/? form

We can use Bats. It's a test framework that enables us to define test suites for bash. In doing so, we can create tests that verify clients meet the interface required by our client specifications. We can test against supported arguments, commands, or even the layout of the output.

Why the f- isn't it right now? Seems kinda obvious, sure.

Because by definition, a "wiki" can be edited by the community.

A collaborative website whose content can be edited by anyone who has access to it.

https://www.wordnik.com/words/wiki

a website that allows visitors to make changes, contributions, or corrections

https://www.merriam-webster.com/dictionary/wiki

If something were to be obvious, it would be that it should be open to the public. Though I have nothing against limiting who has write-access to the wiki, I just want to note why it's not "obvious".

@SethFalco

This comment was marked as off-topic.

@waldyrious
Copy link
Member

I agree that the default-to-open aspect of wikis would be the natural expectation. If GitHub's implementation of the concept makes it hard to detect spam edits (which seems to be the case), and if such edits are prevalent (are they?), then I would agree, albeit reluctantly, that it makes sense to close it down a bit.

As for splitting the unmaintained clients — on the one hand, it pains me to contribute to the march towards obsolescence of software that was perfectly functional in the past, and had the ecosystem around it change from underneath their feet, but I suppose that process is both inevitable and to some degree desirable, lest we block progress in the name of conservatism. Blabbering aside, tl;dr: yeah, I won't object to making the move of abandoning the master branch and de-recognizing clients that haven't updated to support the new branch name.

I would prefer if a broader discussion about compliance to the spec was handled in a separate thread, so that this one could be closed once the two steps I mentioned above are taken. Otherwise I'm afraid this already long thread will drag on for even longer.

@pixelcmtd
Copy link
Member

Given our usual 12 hour waiting period, I'd suggest removing the branch at 17:00 (or 5 PM if you're wrong) UTC, or 2 hours from now

@pixelcmtd
Copy link
Member

`master` Deleted just now by pixelcmtd

I've moved the broken clients from the comment above to the bottom. There are probably more, especially ones that have been archived for a while.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clients Issues pertaining to a particular client or the clients as whole. decision A (possibly breaking) decision regarding tldr-pages content, structure, infrastructure, etc.
Projects
Development

No branches or pull requests

7 participants