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

Restructured, sorted and deleted dead links #4

Open
wants to merge 50 commits into
base: main
Choose a base branch
from

Conversation

metzger100
Copy link
Contributor

@metzger100 metzger100 commented Apr 21, 2022

I restructured the repo and sorted the entries by name.
I also made some categories in the .md-files to make it easier to find stuff.
There is a uncategorized category in ModdingTools which has to be sorted in future.
Also you have to add the links to the files and categories in the new files in README.md!
Thanks for the collection.

Additions of #3 are included.

@metzger100 metzger100 changed the title Restructured, sorted and deleted deadlinks Restructured, sorted and deleted dead links Apr 21, 2022
Copy link
Owner

@jbro129 jbro129 left a comment

Choose a reason for hiding this comment

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

Overall a very good job! I feel like the 'Set Links in the Categories-Index of README.md' in TODO is a must before this pull request can be merged.

README.md Outdated
@@ -6,771 +6,51 @@ A collection of github repositories related to android modding.
- [x] Breakup everything into more categories and adjust accordingly
- [x] Add descriptions for each project
- [ ] Remove unusable projects and duplicates from the list (*in progress*)
- [ ] Make everything look nicer (*in progress*)
- [X] Make everything look nicer
- [ ] Sort stuff Metzger100 didn't know where to put
Copy link
Owner

Choose a reason for hiding this comment

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

Reword to something like "Sort miscellaneous projects"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@metzger100
Copy link
Contributor Author

Well Im not into that Github .md stuff. I don't have the links till you merge, because I only have the links of my Fork and I don't have the ones that are valid when you merge. I only can guess how the links will be,so it might be easier if you merge and simply copy the new links into the README by yourself.

@metzger100
Copy link
Contributor Author

tried to apply links to README now. Hope that they work. And I added your latest Commits to this Pullrequest.

@metzger100
Copy link
Contributor Author

I updated my request with your most recent commits. Could you please consider to merge. I don't want to update your commits for my request forever.

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.

2 participants