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

Backend: Detekt Fixes Part 5 #2617

Merged
merged 16 commits into from
Oct 3, 2024

Conversation

DavidArthurCole
Copy link
Contributor

What

Fixes all instances of the following detektions:

  • ModifierListSpacing
  • NoMultipleSpaces
  • NoSemicolons
  • NoTrailingSpaces
  • PropertyWrapping
  • SpacingAroundComma
  • SpacingAroundCurly
  • SpacingAroundOperators
  • UnnecessaryParenthesesBeforeTrailingLambda

Detektion count down to 630 from 719 at HEAD.

exclude_from_changelog

@hannibal002 hannibal002 added this to the Version 0.28 milestone Sep 29, 2024
Copy link

github-actions bot commented Oct 1, 2024

This pull request has conflicts with the base branch "beta". Please resolve those so we can test out your changes.

@github-actions github-actions bot added the Merge Conflicts There are open merge conflicts with the beta branch. label Oct 1, 2024
Copy link

github-actions bot commented Oct 1, 2024

Conflicts have been resolved! 🎉

@github-actions github-actions bot added Merge Conflicts There are open merge conflicts with the beta branch. and removed Merge Conflicts There are open merge conflicts with the beta branch. labels Oct 1, 2024
Copy link

github-actions bot commented Oct 2, 2024

This pull request has conflicts with the base branch "beta". Please resolve those so we can test out your changes.

Copy link

github-actions bot commented Oct 2, 2024

Conflicts have been resolved! 🎉

@github-actions github-actions bot removed the Merge Conflicts There are open merge conflicts with the beta branch. label Oct 2, 2024
@hannibal002 hannibal002 modified the milestones: Version 0.28, Version 0.27 Oct 2, 2024
@hannibal002 hannibal002 added the Soon This Pull Request will be merged within the next couple of betas label Oct 2, 2024
@hannibal002 hannibal002 merged commit 7efb113 into hannibal002:beta Oct 3, 2024
4 checks passed
@github-actions github-actions bot removed the Soon This Pull Request will be merged within the next couple of betas label Oct 3, 2024
@DavidArthurCole DavidArthurCole deleted the DetektRoundFive branch October 3, 2024 14:19
Tryp0xd pushed a commit to Tryp0xd/SkyHanni that referenced this pull request Oct 3, 2024
* Version 0.27 Beta 13

* Feature: West Village/Dreadfarm/Living Cave features (hannibal002#2616)

Co-authored-by: hannibal2 <24389977+hannibal00212@users.noreply.github.com>

* Fix: Jasper Gemstones not in Powder Mining Filter (hannibal002#2618)

* changed todo

* added parameter `/shcopylocation pathfind`

* Fix a massive bug (hannibal002#2626)

* Fix: Typos in readme (hannibal002#2629)

* Fix: Custom Scoreboard Carnival (hannibal002#2631)

* formatting

* Fix: Add more Pablo patterns (hannibal002#2636)

* Fix: Minister in Calendar Color Reset (hannibal002#2632)

* Fix: Strays Wrongly Counting as Unique (hannibal002#2627)

* Fix: Item trackers saying removed item name from item name (hannibal002#2620)

* fix time format showing empty string for sub 1s

* fixed composter data collection

* Improvement: Add ignored crops for lane features (hannibal002#2622)

* Improvement: Change "efficient miner" to "spread" in mineshaft pity breakdown (hannibal002#2633)

* Backend: Rename keystore file to specify it is for skyhanni (hannibal002#2619)

* Fix: Superpair Data (hannibal002#2566)

Co-authored-by: hannibal2 <24389977+hannibal00212@users.noreply.github.com>
Co-authored-by: CalMWolfs <94038482+CalMWolfs@users.noreply.github.com>

* Improvement: Added cooldown to Carnival NPC click (hannibal002#2628)

* Automatically stating pathfind after clicking the hypixel reminder ch… (hannibal002#2635)

Co-authored-by: hannibal2 <24389977+hannibal00212@users.noreply.github.com>

* Improvement: Pathfind Rendering (hannibal002#2634)

Co-authored-by: hannibal2 <24389977+hannibal00212@users.noreply.github.com>

* Backend: Alert Clock Offsets (hannibal002#2623)

Co-authored-by: hannibal2 <24389977+hannibal00212@users.noreply.github.com>
Co-authored-by: CalMWolfs <cwolfson58@gmail.com>

* fixed typo in broodmother message

* fixed errors when path is zero

* Version 0.27 Beta 14

* Improvement: Slight Touch up for Hoppity Event Summary (hannibal002#2638)

* code cleanup

* Fix: Highlight in Bazaar (hannibal002#2640)

Co-authored-by: hannibal2 <24389977+hannibal00212@users.noreply.github.com>

* Fix: Flare expire warning (hannibal002#2643)

Co-authored-by: hannibal2 <24389977+hannibal00212@users.noreply.github.com>

* Fix: Ender node tracker (hannibal002#2644)

Co-authored-by: hannibal2 <24389977+hannibal00212@users.noreply.github.com>

* Fix: Powder Tracker not tracking items (hannibal002#2645)

* Fix: Preconditions to debug commands (hannibal002#2641)

Co-authored-by: hannibal2 <24389977+hannibal00212@users.noreply.github.com>
Co-authored-by: CalMWolfs <cwolfson58@gmail.com>

* Fix: Dummies and Damage Indicator (hannibal002#2646)

Co-authored-by: hannibal2 <24389977+hannibal00212@users.noreply.github.com>

* fixed wrong feature toggle

* Improvement: Superpair Experimentation data always visible (hannibal002#2647)

Co-authored-by: hannibal2 <24389977+hannibal00212@users.noreply.github.com>
Co-authored-by: CalMWolfs <cwolfson58@gmail.com>

* Backend: Detekt Fixes Part 5 (hannibal002#2617)

---------

Co-authored-by: hannibal2 <24389977+hannibal00212@users.noreply.github.com>
Co-authored-by: MTOnline69 <97001154+MTOnline69@users.noreply.github.com>
Co-authored-by: David Cole <40234707+DavidArthurCole@users.noreply.github.com>
Co-authored-by: CalMWolfs <94038482+CalMWolfs@users.noreply.github.com>
Co-authored-by: J10a1n15 <45315647+j10a1n15@users.noreply.github.com>
Co-authored-by: Luna <me@lunya.dev>
Co-authored-by: martimavocado <39881008+martimavocado@users.noreply.github.com>
Co-authored-by: ILike2WatchMemes <ilike2watchmemes@gmail.com>
Co-authored-by: minhperry <46137516+minhperry@users.noreply.github.com>
Co-authored-by: hannibal2 <24389977+hannibal002@users.noreply.github.com>
Co-authored-by: CalMWolfs <cwolfson58@gmail.com>
Co-authored-by: HiZe <super@hize.be>
Co-authored-by: ItsEmpa <itsempa@users.noreply.github.com>
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.

3 participants