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

General: Clean up Warnings in Code #2971

Closed
wants to merge 1 commit into from

Conversation

Sn0w3y
Copy link
Contributor

@Sn0w3y Sn0w3y commented Jan 16, 2025

image

Copy link

codecov bot commented Jan 16, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ There is a different number of reports uploaded between BASE (79650cb) and HEAD (f5b488b). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (79650cb) HEAD (f5b488b)
java 1 0
Additional details and impacted files
@@              Coverage Diff               @@
##             develop    #2971       +/-   ##
==============================================
- Coverage      56.85%   26.25%   -30.59%     
==============================================
  Files           2281      267     -2014     
  Lines          97624     6844    -90780     
  Branches        7208     1063     -6145     
==============================================
- Hits           55491     1796    -53695     
+ Misses         40047     4962    -35085     
+ Partials        2086       86     -2000     

@Sn0w3y
Copy link
Contributor Author

Sn0w3y commented Jan 17, 2025

Locally Test succeeds..

I even did not make any changes to KEBA :)

image

@Sn0w3y Sn0w3y changed the title Clean up Warnings in Code General: Clean up Warnings in Code Jan 17, 2025
sfeilmeier added a commit that referenced this pull request Jan 20, 2025
@sfeilmeier
Copy link
Contributor

Closing in favour of #2976

@sfeilmeier sfeilmeier closed this Jan 20, 2025
sfeilmeier added a commit that referenced this pull request Jan 20, 2025
- UI
  - use user language instead of browser lang for locale
    -  replace locale being overwritten with browserlang to be using the user lang
    - fix czech locale, `cz` throwing errors for
    - piping (using angular pipes)
    - formatting of numbers
    - replacing it with `cs` -> [Ref](https://github.com/angular/common-builds/blob/main/locales/global/cs.js)
  - Improve Modbus Widget
    - fixes name of channel in ModbusTCP widget (e.g. Ess1SetActivePowerEquals was shown as SetActivePowerEquals/706 in OM)
    - Several ess lead to repeating register names in ModbusTcp widget.
  - Dark mode pick-date improvements
    - Enhancing styles of popover in accordance with dark-mode feature.
  - Add handling for `Controller.IO.Heating.Room`

- Edge
  - Code Cleanup for Java 21
    - Migrate Switch-Case-Expressions to Switch-Case-Statements
    - General code cleanup (using Eclipse "Clean up" tool)
  - EVCS Mennekes: cleanup
  - Cleanup Eclipse warning (#2971)

---------

Co-authored-by: Anas Shetla <[email protected]>
Co-authored-by: Lukas Rieger <[email protected]>
Co-authored-by: Fabian Brandtner <[email protected]>
Co-authored-by: Michael Grill <[email protected]>
Co-authored-by: Sebastian Asen <[email protected]>
Co-authored-by: Christian Lehne <[email protected]>
Co-authored-by: Stefan Feilmeier <[email protected]>
Co-authored-by: Thomas Sicking <[email protected]>
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