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

Update README.md: remove donation links, update maintainer list, announce merge of ripmeapp2 back into ripmeapp #2050

Merged
merged 5 commits into from
Jan 2, 2025

Conversation

metaprime
Copy link
Contributor

@metaprime metaprime commented Jan 2, 2025

Category

This change is exactly one of the following (please change [ ] to [x]) to indicate which:

  • a bug fix (Fix #...)
  • a new Ripper
  • a refactoring
  • a style change/fix
  • a new feature

Description

Hi! I was the primary maintainer of ripmeapp/ripme from about 2016-2018. I'm pleased to see that while that repo was abandoned, a fork of the project continued over here. I'm back to poke around a bit and wanted to sync up the repo as well as make it clear who the primary developer is on the project now. I've also made an announcement post on the subreddit to point to the new active development repo.

I've pushed the main of ripmeapp2/ripme to ripmeapp and I want to make this change as well to both repos to make it clear where active development is happening and who is the current primary developer.

I'm not guaranteeing I'll stick around but I might poke around from time to time. I definitely don't have the time to resume any kind of primary development here. I'm also not necessarily encouraging the resurrection of ripmeapp/ripme over this fork, but if @soloturn wants to accept the maintainer invitation to that repo and merge the forks back together, I'd do what I can to support that. I have a personal (non-work) software-dev-capable computer again so working on a project like this is back on the table.

This change also fixes a Markdown linter style complaint: bulleted lists are preferred to start lines with - to avoid ambiguity with * used for bold/italics.

Testing

(Not applicable to a Markdown-only change.)

Required verification:

  • I've verified that there are no regressions in mvn test (there are no new failures or errors).
  • I've verified that this change works as intended.
    • Downloads all relevant content.
    • Downloads content from multiple pages (as necessary or appropriate).
    • Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.
  • I've verified that this change did not break existing functionality (especially in the Ripper I modified).

Optional but recommended:

  • I've added a unit test to cover my change.

@metaprime metaprime changed the title update readme for ripmeapp2 Update README.md: remove donation links, update maintainer list, announce merge of ripmeapp2 back into ripmeapp Jan 2, 2025
@metaprime
Copy link
Contributor Author

This was originally opened in ripmeapp2#217

@metaprime metaprime merged commit 54ee67e into RipMeApp:main Jan 2, 2025
0 of 4 checks passed
@metaprime metaprime deleted the update-readme-for-ripmeapp2 branch January 2, 2025 07:34
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.

1 participant