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.

Sign up for GitHub

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

Jump to bottom

Change lutX to lutXX #174

Closed
wants to merge 1 commit into from
Closed

Change lutX to lutXX #174

wants to merge 1 commit into from

Conversation

Insomnia1437
Copy link
Contributor

See #172

@gabrielfedel
Copy link
Contributor

Looks good to me!

@jerzyjamroz
Copy link
Contributor

jerzyjamroz commented Aug 23, 2024 •
edited
Loading

@Insomnia1437 , I just noticed that after making the PR #175 as your PR was not listed in the issue #172.
Just reconsider if XX is better than _

@jerzyjamroz jerzyjamroz linked an issue Aug 23, 2024 that may be closed by this pull request
lutX should not be used #172
Closed
@Insomnia1437
Copy link
Contributor Author

Non-alphanumeric characters is a good option! That didn't cross my mind earlier. I can close this PR and see if Michael has further comments on _

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gabrielfedel gabrielfedel gabrielfedel approved these changes

@jerzyjamroz jerzyjamroz Awaiting requested review from jerzyjamroz jerzyjamroz is a code owner

Assignees

@Insomnia1437 Insomnia1437

Labels
fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

lutX should not be used
3 participants
@Insomnia1437 @gabrielfedel @jerzyjamroz