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

Indicate, hide or de-prioritize deprecated projects on the showcase website #301

Open
chalu opened this issue Mar 6, 2024 · 13 comments
Open

Comments

@chalu
Copy link

chalu commented Mar 6, 2024

Projects marked as deprecated on the repo's README are not so reflected on the showcase website.

This leads to inconsistency on what is being communicated. and may make it somewhat difficult to decide between two similar projects if you can quickly tell that one is deprecated

@rotimi-best
Copy link
Contributor

rotimi-best commented Mar 7, 2024

I agree, I just implemented a change that checks for archived and deprecated repos then shows it as a chip on the UI.

Ideally we should have like a filter on the projects list to hide archived or deprecated repos.

image

@kaf-lamed-beyt what do you think? I am working on a PR with this update.

@kaf-lamed-beyt
Copy link
Collaborator

LGTM!

@acekyd, @D-Lite is the color choice okay?

@acekyd
Copy link
Owner

acekyd commented Mar 7, 2024

The color choice looks good. On second thought however, a good thing to note is that the criteria for setting projects as deprecated were that the last commit was over 2 years ago (If I remember correctly). This process was also done manually - hopefully there could be an easy way to implement it in future.

However I'd like to change the intent to something else from deprecated - maybe like not actively maintained or something shorter - cos some of them still work and deprecated might not be the fitting term.

@D-Lite
Copy link
Collaborator

D-Lite commented Mar 7, 2024

What do you think of unmaintained or inactive? @acekyd @kaf-lamed-beyt @chalu

@kaf-lamed-beyt
Copy link
Collaborator

"Inactive" has a nice ring to it.👌🏽

@rotimi-best
Copy link
Contributor

+1 for "Inactive"

In my PR should I rename Deprecated in the README to Inactive?

@acekyd
Copy link
Owner

acekyd commented Mar 7, 2024

Yes @rotimi-best. Inactive does sound good.

@chalu
Copy link
Author

chalu commented Mar 7, 2024

Inactive sounds good to me as well

@rotimi-best
Copy link
Contributor

@acekyd @kaf-lamed-beyt @chalu.

Here is my checklist for completion

  1. Rename Deprecated to Inactive. docs: rename Deprecated to Inactive #310
  2. Implement the parsing of the README on main and then rendering of the tag on each project.

@kaf-lamed-beyt
Copy link
Collaborator

Hi @rotimi-best, I think, it'll be best if we can sort of automate this, just like @acekyd suggested, instead of manually typing it.

One way to go about this would be to use the GitHub Commit REST API to get the last commit date, and compare if it is in the range @acekyd mentioned — 2 years in this case.

The payload's response includes the date, we can use that.

If you're not going to be able to work on it for now, that's absolutely cool... perhaps @acekyd can merge this PR, then we'd look for a way to work on it internally.

@rotimi-best
Copy link
Contributor

@kaf-lamed-beyt It shouldn't be complicated, I can def work on it.

If I am implementing the automated approach of checking if it is archived or inactive, should I just remove the text in the README that mentions whether it is "Inactive" or "Archived" so it doesn't confuse anyone in the future as we add more projects?

@kaf-lamed-beyt
Copy link
Collaborator

Yes. you should remove it. Since It'll be automated now.

If you have any questions, let me know, I'd be happy to help.

@acekyd
Copy link
Owner

acekyd commented Mar 9, 2024

You can proceed to test this on any branch now since the data source now uses the local file system (#314) now @rotimi-best - for the automation check, lets keep the processing time in mind so we don't lock up the main thread and make it all slower running API requests for 200+ projects.

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

No branches or pull requests

5 participants