README review#165
Open
TheMrIron2 wants to merge 3 commits into
Open
Conversation
Author
|
README overhauled. Read the latest commit's details for a summary. |
- "Current Status" was out of date, and its underlying model is wrong. Several of the claims listed are stale - hostname lookup is not actually "missing", the filesystem header has moved to `ppu/include`, and so on. PSL1GHT has lots of legacy code, and much of it has not been tested in years; it will be impossible to give an honest status until the SDK matures and settles post-modernisation and likely until more maintainers are on-board. Instead, it's been reframed to point towards a live source of documentation (DeepWiki) and not try to give a broad but untested feature overview. - Credits moved to the end to focus on repository status. - Change wording surrounding the PS3 toolchain (eg. "at the moment only one toolchain does so") to more accurately reflect current PS3DEV. - Remove old claim that Python 2.x is required for `fself.py`, `sfo.py` and `pkg.py`. These have since been refactored into C/C++.
- add `sh` to the markdown code blocks and add backticks where missing - fix http://github.com to https - tighten some phrasing - add link to contributors graph
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
README has not been updated in years - it's stale by now. For example @bucanero you may confirm if the Python requirement mentioned here is outdated, given you've worked on new scripts to replace it recently.
Rather than just commit this one edit, I opened this as a PR to review it and so we can then cleanly update it in one squashed merge.