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

spec_includes/ -> spec/, spec -> spec/spec #2422

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

Conversation

Dragorn421
Copy link
Collaborator

No description provided.

Makefile Show resolved Hide resolved
Copy link
Collaborator

@fig02 fig02 Jan 13, 2025

Choose a reason for hiding this comment

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

This really should have been a comment for #2399

But _gc_ique is pretty confusing. I figured after some more thought that it means "gc AND ique", but seeing it alongside _ique_n64 makes it seem like its saying there is a gamecube and n64 ique variant lol

Im not sure how to address this other than putting the word "and". Also understand if this conversation/change should happen elsewhere.

@fig02
Copy link
Collaborator

fig02 commented Jan 16, 2025

was hoping to get a bit more than 2 approvals on this kind of change, but I guess there hasnt been any negative comments either.
If people are indifferent thats fine, but try to approve if youre okay with it!

Will merge in 48-ish hours if there is no more activity.

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.

5 participants