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

Made preview display same format as output #112

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

flashrun24
Copy link
Contributor

What I did:

  • Made the preview format the same as the output format, addresses Preview is always hex #109
  • Made lowercase hex into a format to make the code more readable
  • Updated flake lock
  • Bumped GitHub actions to latest

Video of it working:

https://github.com/user-attachments/assets/8cb7b184-28c9-4d3c-8679-61744d1a523d
Please let me know if I need to do or change anything.

@flashrun24 flashrun24 marked this pull request as ready for review February 20, 2025 05:59
@flashrun24
Copy link
Contributor Author

Also, I am not used to contributing to projects with forks so I am not sure what I did but for some reason it has the commits from the previous PR I made, I don't think this affects anything though. Just let me know if its a problem.

@vaxerski
Copy link
Member

uhh, it has some stuff removed.

@flashrun24
Copy link
Contributor Author

uhh, it has some stuff removed.

Resolved the conflict, I removed the --lowercase-hex since it makes more sense to me for lowercase hex to be a format, let me know if I should undo this.

@vaxerski
Copy link
Member

it should be delegated to a separate MR as it's completely unrelated to this

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