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

Repositories overview and actions #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

sheplu
Copy link
Member

@sheplu sheplu commented Feb 29, 2024

Here is the list of all repositories that are currently marked as "active" in the three organizations (Express, Pillarjs, JShttp).
Some / a good numbers of them are empty or count for almost no downloads.

This PR is to validate and agree between TC members which repositories can be archived or deleted completely (only if the repository was empty)

Some repositories are representing a small amont of download per week but still suffisant to discuss if we want to maintain them or deprecate them.

After all repositories are deleted or archived, it will be necessary to deprecate the packages on npmjs.org

@sheplu
Copy link
Member Author

sheplu commented Feb 29, 2024

if / when we are all good with the decisions in this PR (after change and discussion if needed) I will archive/delete all repositories

@expressjs/express-tc

@wesleytodd
Copy link
Member

While I think this is a good first step, there are so many things we need to align on across all the repos. I am unsure a markdown table will be enough will it?

@sheplu
Copy link
Member Author

sheplu commented Feb 29, 2024

While I think this is a good first step, there are so many things we need to align on across all the repos. I am unsure a markdown table will be enough will it?

The point of this PR is just to do a first step of cleaning, not all actions. But this would allow us to agree on which repos we are removing (empty ones), which one we are archiving (not used, with code not deployed), and which one we keep.

It is just for us to agree on this list, and then after that we will consolidate a list of repositories we will have to work on.

@wesleytodd
Copy link
Member

Ok awesome. Sounds good as long as we are going to do something a bit more "high fidelity" for all that other stuff to make it easier to read.

@sheplu
Copy link
Member Author

sheplu commented Feb 29, 2024

I think for a more global approach for all packages we should

  • outline the expectations at a global scope
  • write a roadmap for a package in its own repository (to have an easy tracking)
  • reference all roadmaps in one issue here to have a global tracking

@wesleytodd
Copy link
Member

We could use a GH project board for that. They are fairly easy to setup if we wanted. I think across the orgs doesn't quite work right unless they are in an enterprise account but we could just setup one for each org and link to those in here?

@sheplu
Copy link
Member Author

sheplu commented Feb 29, 2024

this would also work, I never really tried that feature :)

| [qs-strict](https://github.com/pillarjs/qs-strict) | 0 (1) | 9 years old | archive |
| [request](https://github.com/pillarjs/request) | N/A | where? | archive |
| [resolve-path](https://github.com/pillarjs/resolve-path) | 0.8 | 6 years old | update and keep OR archive |
| [router](https://github.com/pillarjs/router) | 0.6 | - | update and keep OR archive |
Copy link
Member

@blakeembrey blakeembrey Mar 7, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the core router in express 5 so should be keep (unless we're refactoring code back into express).

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh yeah for sure! This is absolutely keep. The main work is on the 2.0 branch. I was planning on (as I volunteered to be the project captain) to get that merged and make it more clear what is going on in there. I just have not yet had a chance to do that.

Copy link
Member

@inigomarquinez inigomarquinez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've added some missing repositories.

| [timeout](https://github.com/expressjs/timeout) | 0.22 | 7 years old | update and keep OR archive |
| [urlrouter](https://github.com/expressjs/urlrouter) | 0 (142) | 10 years old | archive |
| [vhost](https://github.com/expressjs/vhost) | 0.12 | 8 years old | update and keep OR archive |
| [vhostess](https://github.com/expressjs/vhostess) | 0 (22) | 10 years old | archive |
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We're missing 2 repos here:

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

they are already archived so not part of the list

| [ssl-redirect](https://github.com/pillarjs/ssl-redirect) | 0 (43) | 9 years old | archive |
| [templation](https://github.com/pillarjs/templation) | 0 (27) | 9 years old | |
| [understanding-csrf](https://github.com/pillarjs/understanding-csrf) | N/A | reuse the doc | archive |
| [view](https://github.com/pillarjs/views) | N/A | - | remove |
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We're missing 1 repo here:

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

they are already archived so not part of the list

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

Successfully merging this pull request may close these issues.

4 participants