Skip to content

Jv 880 v1.00 hashes#62

Closed
akse0435 wants to merge 1 commit intojcmoyer:masterfrom
akse0435:JV-880-v1.00-hashes
Closed

Jv 880 v1.00 hashes#62
akse0435 wants to merge 1 commit intojcmoyer:masterfrom
akse0435:JV-880-v1.00-hashes

Conversation

@akse0435
Copy link
Copy Markdown

This adds hashes for JV-880 version 1.0.0. ROM2 in version 1.0.1 actually had the part number for version 1.0.0, R15209386, writen in the code even though the hash was the right one for version 1.0.1, so I changed the part number to the right one, R15209481, and used R15209386 with the right has in version 1.0.0. I also added version number for the SCC-1A set, it's v1.30.

@jcmoyer
Copy link
Copy Markdown
Owner

jcmoyer commented Feb 10, 2026

Thanks! Would you mind squashing the commits?

Added JV-880 v1.00 hashes and made a few corrections

Reverted change of version numbers for JV-880, since there appears to be confusion around the internet about how to write them.
@akse0435 akse0435 force-pushed the JV-880-v1.00-hashes branch from 307af79 to 3a24cac Compare February 10, 2026 20:04
@akse0435
Copy link
Copy Markdown
Author

Should be done now. I've never tried squashing before, so I hope I did it correctly.
Sorry about the multiple piull requests by the way, I only discovered this after submiting the last one.

@jcmoyer
Copy link
Copy Markdown
Owner

jcmoyer commented Feb 11, 2026

LGTM, thanks. One thing I would suggest for next time is to discard any commit messages for changes that you reverted. Here I would use git rebase -i HEAD~2 to select the latest two commits and change the second commit where you reverted the JV-880 version numbers from pick to fixup.

I've manually merged this one outside of github with an edited commit message.

jcmoyer added a commit that referenced this pull request Feb 11, 2026
@jcmoyer jcmoyer closed this Feb 11, 2026
@akse0435 akse0435 deleted the JV-880-v1.00-hashes branch February 13, 2026 17:50
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