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

Rename bitmaps into bmp, to prevent misleading #2460

Merged
merged 1 commit into from
Jan 3, 2025

Conversation

zxkmm
Copy link
Contributor

@zxkmm zxkmm commented Dec 31, 2024

  • rename bitmaps folder into bmps (it's a mistake from someone who made it), bmp isn't bitmap
  • In short, on portapack's graphic thing, only splash/sstv/warning modal using bmp, others is using bitmap, the two files is bmp instead of bitmap

@zxkmm zxkmm closed this Dec 31, 2024
@zxkmm zxkmm force-pushed the bitmap_and_bmp_tune branch from 6356d74 to 4c18b80 Compare December 31, 2024 08:41
@zxkmm zxkmm reopened this Dec 31, 2024
@zxkmm zxkmm changed the title bitmap and bmp things tune Rename bitmaps into bmp, to prevent misleading Jan 1, 2025
@zxkmm zxkmm marked this pull request as ready for review January 1, 2025 02:34
@zxkmm zxkmm enabled auto-merge (squash) January 1, 2025 02:35
@zxkmm zxkmm marked this pull request as draft January 1, 2025 12:39
auto-merge was automatically disabled January 1, 2025 12:39

Pull request was converted to draft

@zxkmm zxkmm marked this pull request as ready for review January 3, 2025 04:26
@zxkmm
Copy link
Contributor Author

zxkmm commented Jan 3, 2025

I originally wanted this to be have one go with the key copier app, but after talking with community, i believe adding that app is useless. So this PR would be ready as is.

Copy link
Member

@gullradriel gullradriel left a comment

Choose a reason for hiding this comment

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

I see. I agree it could be misleading.
It's a minor change anyway.

@gullradriel gullradriel merged commit 9788949 into portapack-mayhem:next Jan 3, 2025
6 checks passed
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