PPPwn icon indicating copy to clipboard operation
PPPwn copied to clipboard

Clarify READme

Open fabianlanza opened this issue 1 year ago • 6 comments

I've updated the README to include information about how additional firmware versions can be supported through pull requests. I've also clarified that 'FW' stands for 'Firmware' throughout the documentation for better understanding. This change aims to improve clarity and encourage collaboration from the community. Thank you for considering my contribution!

fabianlanza avatar May 03 '24 00:05 fabianlanza

No need for the FW clarification though. I think everybody knows that who understands a bit of software-side, especially in Github

Fastball2880 avatar May 03 '24 00:05 Fastball2880

Change the "pull request" to PR and probably remove the note too, if needed

Fastball2880 avatar May 03 '24 00:05 Fastball2880

haha believe me, no everyone knows what FW stands for

fabianlanza avatar May 03 '24 00:05 fabianlanza

haha believe me, no everyone knows what FW stands for

I understand, but I think reading from the README. End-users and developers understand why they go to this repository, and what type of steps they need to do. Otherwise, if they don't, this repository isn't designed for them.

Like, they know they want to see the exploit and see if it's compatible by their version of PS4. I think they understand how they check that right?

Fastball2880 avatar May 03 '24 01:05 Fastball2880

haha believe me, no everyone knows what FW stands for

I understand, but I think reading from the README. End-users and developers understand why they go to this repository, and what type of steps they need to do. Otherwise, if they don't, this repository isn't designed for them.

Like, they know they want to see the exploit and see if it's compatible by their version of PS4. I think they understand how they check that right?

I appreciate your perspective, and I understand that the primary audience for this repository is likely to be developers and enthusiasts familiar with firmware and software terminology. However, considering that the README serves as the first point of contact for users, including those who might be less experienced, providing clear explanations can enhance accessibility and usability for all potential users. Clarifying the meaning of 'FW' as 'Firmware' not only aids those who may not be as familiar with the term but also demonstrates a commitment to inclusivity and user-friendliness. Ultimately, the goal is to make the repository as accessible and helpful as possible to a wide range of users. Thank you for considering this perspective.

fabianlanza avatar May 03 '24 01:05 fabianlanza

haha believe me, no everyone knows what FW stands for

I understand, but I think reading from the README. End-users and developers understand why they go to this repository, and what type of steps they need to do. Otherwise, if they don't, this repository isn't designed for them. Like, they know they want to see the exploit and see if it's compatible by their version of PS4. I think they understand how they check that right?

I appreciate your perspective, and I understand that the primary audience for this repository is likely to be developers and enthusiasts familiar with firmware and software terminology. However, considering that the README serves as the first point of contact for users, including those who might be less experienced, providing clear explanations can enhance accessibility and usability for all potential users. Clarifying the meaning of 'FW' as 'Firmware' not only aids those who may not be as familiar with the term but also demonstrates a commitment to inclusivity and user-friendliness. Ultimately, the goal is to make the repository as accessible and helpful as possible to a wide range of users. Thank you for considering this perspective.

Very good point actually. Surely that the terminology will serve some idea for some people who wants to understand how exploit works in the first place from this source code.

But for the general population, that user-friendliness you mentioned will actually go to the established homebrew such as GoldHEN (as it will be released shortly for end-users). However when it comes to these homebrews in general, some users may need an understanding of terminology from source code or homebrew: including FW, userland, pwned, etc.

Overall, you can brainstorm more on how you can add more inclusivity for beginner developers, or people who want to experiment. This makes people read more into instructions and provide a friendly environment on how they read it too. (see more in Arch Linux wikis)

Fastball2880 avatar May 03 '24 02:05 Fastball2880

Anyone who doesn't understand what FW or PR is shouldnt be making pull requests anyways

TheOfficialFloW avatar May 03 '24 06:05 TheOfficialFloW

Lol, I never said I didn't understand what it is. I made my point on the messages from before, but thanks anyway

On Fri, May 3, 2024 at 12:52 AM Andy Nguyen @.***> wrote:

Anyone who doesn't understand what FW or PR is shouldnt be making pull requests anyways

— Reply to this email directly, view it on GitHub https://github.com/TheOfficialFloW/PPPwn/pull/41#issuecomment-2092409725, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOKF6L6NSI7XH6BUDNXO52TZAMXZDAVCNFSM6AAAAABHEVLHHWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJSGQYDSNZSGU . You are receiving this because you authored the thread.Message ID: @.***>

fabianlanza avatar May 03 '24 06:05 fabianlanza

I didn't say you wouldn't understand duh..

TheOfficialFloW avatar May 03 '24 07:05 TheOfficialFloW

Alright haha since you said PR are welcome I was just trying to make the readme more understandable and friendly since thats what it is used for.

El vie, 3 de may. de 2024 1:09 a. m., Andy Nguyen @.***> escribió:

I didn't say you wouldn't understand duh..

— Reply to this email directly, view it on GitHub https://github.com/TheOfficialFloW/PPPwn/pull/41#issuecomment-2092434119, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOKF6L4EIPHAFCBHPNLJWYDZAMZZHAVCNFSM6AAAAABHEVLHHWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJSGQZTIMJRHE . You are receiving this because you authored the thread.Message ID: @.***>

fabianlanza avatar May 03 '24 07:05 fabianlanza

PRs for new firmware supports. That only developers are able to do anyways.

TheOfficialFloW avatar May 03 '24 07:05 TheOfficialFloW

True true

El vie, 3 de may. de 2024 1:25 a. m., Andy Nguyen @.***> escribió:

PRs for new firmware supports. That only developers are able to do anyways.

— Reply to this email directly, view it on GitHub https://github.com/TheOfficialFloW/PPPwn/pull/41#issuecomment-2092455882, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOKF6L7KZ6AJ3I7EMMOSUELZAM3YFAVCNFSM6AAAAABHEVLHHWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJSGQ2TKOBYGI . You are receiving this because you authored the thread.Message ID: @.***>

fabianlanza avatar May 03 '24 07:05 fabianlanza