Ruff-only pre-commit hooks #855
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
gantoine
force-pushed
the
ruff-precommit-hooks
branch
from
fa539d8
to
c354b3f
Compare
gantoine
force-pushed
the
ruff-precommit-hooks
branch
from
c354b3f
to
880345f
Compare
zurdi15
approved these changes
May 19, 2024
Merged
spiceratops
referenced
this pull request
in spiceratops/k8s-gitops
Jun 8, 2024
This PR contains the following updates: | Package | Update | Change | |---|---|---| | [rommapp/romm](https://github.com/rommapp/romm) | minor | `3.1.0` -> `3.2.0` | --- > [!WARNING] > Some dependencies could not be looked up. Check the Dependency Dashboard for more information. --- ### Release Notes <details> <summary>rommapp/romm (rommapp/romm)</summary> ### [`v3.2.0`](https://github.com/rommapp/romm/releases/tag/3.2.0) [Compare Source](https://github.com/rommapp/romm/compare/3.1.0...3.2.0) ### BIOS file support We've added the ability to mount (or upload) your BIOS/firmware files for any of the supported platforms. On top of that, when playing in browser with our EmulatorJS integration, you'll now be able to select both a BIOS file to load, and a core (when the system supports more then 1 core). Refer to the EmulatorJS for the complete list of available cores for each system. ### Sharable & private notes When viewing a game, you might notice a new tab called "Notes"; this is your notepad, a blank space to jot down just about anything. With full Markdown support, you have complete control over the style and layout of your note. You're also able to make the note "public" (using the little eye icon next to the pencil), that is visible to every logged-in user on your instance. And if they do the same, you'll get access to their note, for easy sharing of tips and tricks. ||| |---|---| ||| ### Performance improvements On collections of 1000+ games/platforms with 250+ roms, we've measured a 10x to 20x improvement in load times. RomM should now feel faster regardless of the size of your collection, with smoother gallery navigation, faster loading of games on scroll, and real pagination support in the list view. #### What's Changed - Games notes by [@​gantoine](https://github.com/gantoine) in [https://github.com/rommapp/romm/pull/828](https://github.com/rommapp/romm/pull/828) - Copy download link dialog by [@​zurdi15](https://github.com/zurdi15) in [https://github.com/rommapp/romm/pull/846](https://github.com/rommapp/romm/pull/846) - Firmware/BIOS support by [@​gantoine](https://github.com/gantoine) in [https://github.com/rommapp/romm/pull/830](https://github.com/rommapp/romm/pull/830) - Upgraded front packages and adapted code by [@​zurdi15](https://github.com/zurdi15) in [https://github.com/rommapp/romm/pull/847](https://github.com/rommapp/romm/pull/847) - Python package changes by [@​gantoine](https://github.com/gantoine) in [https://github.com/rommapp/romm/pull/844](https://github.com/rommapp/romm/pull/844) - Fixed gallery play button for not supported platforms by [@​zurdi15](https://github.com/zurdi15) in [https://github.com/rommapp/romm/pull/848](https://github.com/rommapp/romm/pull/848) - Fixed updating user images and ignore firmware files based on config by [@​gantoine](https://github.com/gantoine) in [https://github.com/rommapp/romm/pull/851](https://github.com/rommapp/romm/pull/851) - Purge platforms correctly on scan by [@​gantoine](https://github.com/gantoine) in [https://github.com/rommapp/romm/pull/858](https://github.com/rommapp/romm/pull/858) - Ruff-only pre-commit hooks by [@​gantoine](https://github.com/gantoine) in [https://github.com/rommapp/romm/pull/855](https://github.com/rommapp/romm/pull/855) - Added amiibo platform icon by [@​Casuallynoted](https://github.com/Casuallynoted) in [https://github.com/rommapp/romm/pull/869](https://github.com/rommapp/romm/pull/869) - Fixed emulatorjs game loading when game have non-ascii characters in the name by [@​vincywindy](https://github.com/vincywindy) in [https://github.com/rommapp/romm/pull/872](https://github.com/rommapp/romm/pull/872) - Fix search by ID for mobygames by [@​gantoine](https://github.com/gantoine) in [https://github.com/rommapp/romm/pull/863](https://github.com/rommapp/romm/pull/863) - Platform fetch performance improvements by [@​gantoine](https://github.com/gantoine) in [https://github.com/rommapp/romm/pull/857](https://github.com/rommapp/romm/pull/857) - Fixed uploading images for roms and users by [@​gantoine](https://github.com/gantoine) in [https://github.com/rommapp/romm/pull/874](https://github.com/rommapp/romm/pull/874) - Fixed rom download by [@​vincywindy](https://github.com/vincywindy) in [https://github.com/rommapp/romm/pull/876](https://github.com/rommapp/romm/pull/876) - Improved general performance (10x faster load at home page and galleries) by [@​gantoine](https://github.com/gantoine) in [https://github.com/rommapp/romm/pull/875](https://github.com/rommapp/romm/pull/875) - Fixed scan view selectors by [@​zurdi15](https://github.com/zurdi15) in [https://github.com/rommapp/romm/pull/878](https://github.com/rommapp/romm/pull/878) - Whats new link fixed by [@​zurdi15](https://github.com/zurdi15) in [https://github.com/rommapp/romm/pull/880](https://github.com/rommapp/romm/pull/880) - Only scan platform when selected by [@​gantoine](https://github.com/gantoine) in [https://github.com/rommapp/romm/pull/882](https://github.com/rommapp/romm/pull/882) - Improved performance of Rom DB fetch by [@​gantoine](https://github.com/gantoine) in [https://github.com/rommapp/romm/pull/883](https://github.com/rommapp/romm/pull/883) #### New Contributors - [@​vincywindy](https://github.com/vincywindy) made their first contribution in [https://github.com/rommapp/romm/pull/872](https://github.com/rommapp/romm/pull/872) - [@​Deekerman](https://github.com/Deekerman) a special shoutout for being a right PITA </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zODIuNCIsInVwZGF0ZWRJblZlciI6IjM3LjM4Mi40IiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJyZW5vdmF0ZS9jb250YWluZXIiLCJ0eXBlL21pbm9yIl19-->
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds pre-commit hooks for general file cleanup and formatting with ruff. All the changes are purely cleanup and shouldn't have changed any business logic. There's also commented out config for running mypy, which will need more work to get working, so will happen in a separate PR.
After merging, run
pip/brew install pre-committo install the pre-commit package globally.