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

Hide lgpt_ prefix #165

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

mrtnvgr
Copy link

@mrtnvgr mrtnvgr commented Nov 1, 2024

No description provided.

@mrtnvgr mrtnvgr marked this pull request as draft November 1, 2024 03:57
@mrtnvgr mrtnvgr marked this pull request as ready for review November 1, 2024 04:30
@mrtnvgr mrtnvgr marked this pull request as draft November 1, 2024 14:17
@mrtnvgr mrtnvgr changed the title Add ability to hide lgpt_ prefixes Hide lgpt_ prefix Nov 2, 2024
@mrtnvgr mrtnvgr marked this pull request as ready for review November 2, 2024 09:35
@mrtnvgr mrtnvgr marked this pull request as draft November 2, 2024 12:16
@mrtnvgr mrtnvgr marked this pull request as ready for review November 2, 2024 12:20
@djdiskmachine
Copy link
Owner

Thank you for contributing to Little Piggy Tracker! For fixes, please include a summary of the changes and any related issue. For features, please include a detailed description of the new feature and how it works.

All PR:s are run through line-by-line clang-format and builds for all supported platforms. clang-format can be applied locally on commit by installing pre-commit from repo root: cp pre-commit .git/hooks/

Fixes # (issue)
Type of change

Please delete option that isn't relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions on how to reproduce.

Test Configuration:

Hardware:
OS version:

Checklist:

I have performed a self-review of my code
I have commented particularly in hard-to-understand areas
I have updated documentation regarding my changes
My changes generate no new warnings

@mrtnvgr
Copy link
Author

mrtnvgr commented Nov 11, 2024

Type of change: New feature (non-breaking change which adds functionality)
How Has This Been Tested?: the change visuals are correct

Test Configuration:

Hardware: Linux x86, PSP E1000
OS version: NixOS unstable, last PSP OS version

Checklist:

  • I have performed a self-review of my code
  • I have commented particularly in hard-to-understand areas (i think that there are no hard-to-understand areas)
  • I have updated documentation regarding my changes
  • My changes generate no new warnings (i'm not sure, don't know how to check this sorry)

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