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

GameDB: some fixes #11508

Merged
merged 2 commits into from
Jul 7, 2024
Merged

GameDB: some fixes #11508

merged 2 commits into from
Jul 7, 2024

Conversation

nishinji
Copy link
Contributor

@nishinji nishinji commented Jul 6, 2024

Description of Changes

Enclose the strings by quotation marks
Remove (some) symbols from name-sort that are not read aloud
This is for NTSC-J

Rationale behind Changes

More consistent code rules
Sorting should be determined by actual pronounce

Suggested Testing Steps

It is tested on my pc

@github-actions github-actions bot added the GameDB label Jul 6, 2024
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thank you for submitting a contribution to PCSX2

As this is your first pull request, please be aware of the contributing guidelines.

Additionally, as per recent changes in GitHub Actions, your pull request will need to be approved by a maintainer before GitHub Actions can run against it. You can find more information about this change here.

Please be patient until this happens. In the meantime if you'd like to confirm the builds are passing, you have the option of opening a PR on your own fork, just make sure your fork's master branch is up to date!

@refractionpcsx2
Copy link
Member

Not sure why you're removing brackets from the types of disc? We've pretty consistently done that through the whole DB, but you removed a couple? Doesn't make sense. I'd rather they were there, tbh,

@nishinji
Copy link
Contributor Author

nishinji commented Jul 7, 2024

Not sure why you're removing brackets from the types of disc? We've pretty consistently done that through the whole DB, but you removed a couple? Doesn't make sense. I'd rather they were there, tbh,

Alright. I'll revert it when I have time.

Copy link
Contributor

@lightningterror lightningterror left a comment

Choose a reason for hiding this comment

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

Seems like something got messed up, I can't see the diff since there's a lot of changes.
See https://github.com/PCSX2/pcsx2/actions/runs/9825954359/job/27131965803?pr=11508#step:5:1

@nishinji
Copy link
Contributor Author

nishinji commented Jul 7, 2024

Git is working property I expect
image
Sorry my typo. I regret my folly of using the master branch...

Copy link
Contributor

@lightningterror lightningterror left a comment

Choose a reason for hiding this comment

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

Looks good now.

@lightningterror lightningterror merged commit c127d23 into PCSX2:master Jul 7, 2024
13 checks passed
SternXD pushed a commit to EmulationCollective/XBSX2 that referenced this pull request Jul 18, 2024
Enclose the strings by quotation marks.
Remove (some) symbols from name-sort that are not read aloud.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants