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

Add feature table matrix to Firmwares.md #125

Open
wants to merge 1 commit into
base: incoming
Choose a base branch
from

Conversation

jordanmalecki
Copy link

@jordanmalecki jordanmalecki commented Nov 28, 2023

I wanted to view the firmware in a more distinguishable format.
I'm opening a pull request in case others do too.

@RogueMaster
Copy link
Contributor

Squachware having checks that RM doesn't have may be a sign your table may be missing some information...

@jordanmalecki
Copy link
Author

Good eye.
I ran out of time to give it a proper review. Maybe a bit too hasty with the request. Tried to share before I got tied down.

Ill give it a once over when I'm free

@xMasterX
Copy link
Contributor

xMasterX commented Nov 29, 2023

Misinformation in those categories:

Built-in Name Changer - Present in Unleashed, XFW and other non abandoned forks

Cracking RFID Passwords via Dictionary Attack - Present in Unleashed, XFW and other non abandoned forks

Other Small Tweaks and Changes - What is that? Why some of them has it and some not?

Different Keyboard Layouts for Duckyscripts - Present in OFW and all forks except abandoned ones that you added in that chart for some reason

Also I made updates to this fw comparison but DJ is not merging his incoming branch into main, so this will conflict

@jordanmalecki
Copy link
Author

Should I just close this one?
Not sure if the ensuing conflicts complicate things

@djsime1
Copy link
Owner

djsime1 commented Dec 4, 2023

Also I made updates to this fw comparison but DJ is not merging his incoming branch into main, so this will conflict

Ah shit you're right, completely forgot to do that after the last bulk PR merge-athon.

Should I just close this one?
Not sure if the ensuing conflicts complicate things

I think this PR can remain open to fixes until it's correct. I really dig this easy comparison table and would probably place it at the top of the file instead of the bottom.

@Crashinsane
Copy link

Crashinsane commented Dec 6, 2023 via email

@djsime1 djsime1 changed the base branch from main to update-66 December 23, 2023 23:18
@RogueMaster
Copy link
Contributor

Built-in Name Changer - Present in Unleashed, XFW and other non abandoned forks

Cracking RFID Passwords via Dictionary Attack - Present in Unleashed, XFW and other non abandoned forks

Other Small Tweaks and Changes - What is that? Why some of them has it and some not?

Different Keyboard Layouts for Duckyscripts - Present in OFW and all forks except abandoned ones that you added in that chart for some reason

RM has all 4 of these items too...

@djsime1 djsime1 changed the base branch from update-66 to incoming March 11, 2024 20:51
@djsime1
Copy link
Owner

djsime1 commented Mar 19, 2024

This is the last PR to merge before squashing everything into update 68, I don't use any custom firmwares so I don't know the differences of each myself. I really like the table, but this document now also needs to include the new Momentum firmware. I'll commit any changes I notice, but would appreciate if someone else also reviews this before I merge.

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.

6 participants