GladysAssistant / Gladys

A privacy-first, open-source home assistant
https://gladysassistant.com
Apache License 2.0
2.57k stars 278 forks source link

Update mqtt version #2053

Closed McFlyPartages closed 3 months ago

McFlyPartages commented 4 months ago

Pull Request check-list

To ensure your Pull Request can be accepted as fast as possible, make sure to review and check all of these items:

NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.

Description of change

Just update version of MQTT Brocker v2.0.15 to 2.0.18.

https://community.gladysassistant.com/t/mosquitto-v2-0-15-au-lieu-de-latest-ou-v2-0-18/8835

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.38%. Comparing base (8f13438) to head (3b3bb68).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #2053 +/- ## ======================================= Coverage 98.38% 98.38% ======================================= Files 844 844 Lines 13753 13753 ======================================= Hits 13531 13531 Misses 222 222 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

relativeci[bot] commented 4 months ago

Job #2464: Bundle Size — 10.04MiB (0%).

3b3bb68(current) vs 8f13438 master#2463(baseline)

[!WARNING] Bundle contains 3 duplicate packages – View duplicate packages

Bundle metrics  no changes
                 Current
Job #2464
     Baseline
Job #2463
No change  Initial JS 5.38MiB 5.38MiB
No change  Initial CSS 303.21KiB 303.21KiB
No change  Cache Invalidation 0% 0%
No change  Chunks 51 51
No change  Assets 170 170
No change  Modules 1459 1459
No change  Duplicate Modules 20 20
No change  Duplicate Code 0.78% 0.78%
No change  Packages 124 124
No change  Duplicate Packages 3 3

Bundle size by type  no changes
|            |       Current
[Job #2464](https://app.relative-ci.com/projects/PUROh8FAVkDKmpUrqr4u/jobs/2464-sDdYWs30fnAz8R3NF0xC?utm_source=github&utm_medium=pr-report "View job report") |      Baseline
[Job #2463](https://app.relative-ci.com/projects/PUROh8FAVkDKmpUrqr4u/jobs/2463-EjUp1628OyVtqTuutwwR?utm_source=github&utm_medium=pr-report "View baseline job report") | |:--|--:|--:| | Not changed  [JS](https://app.relative-ci.com/projects/PUROh8FAVkDKmpUrqr4u/jobs/2464-sDdYWs30fnAz8R3NF0xC/assets?ba=%7B%22filters%22%3A%22ft.CSS-0_ft.JS-1_ft.IMG-0_ft.MEDIA-0_ft.FONT-0_ft.HTML-0_ft.OTHER-0%22%7D "View JS assets") | `7.14MiB` | `7.14MiB` | | Not changed  [IMG](https://app.relative-ci.com/projects/PUROh8FAVkDKmpUrqr4u/jobs/2464-sDdYWs30fnAz8R3NF0xC/assets?ba=%7B%22filters%22%3A%22ft.CSS-0_ft.JS-0_ft.IMG-1_ft.MEDIA-0_ft.FONT-0_ft.HTML-0_ft.OTHER-0%22%7D "View IMG assets") | `2.46MiB` | `2.46MiB` | | Not changed  [CSS](https://app.relative-ci.com/projects/PUROh8FAVkDKmpUrqr4u/jobs/2464-sDdYWs30fnAz8R3NF0xC/assets?ba=%7B%22filters%22%3A%22ft.CSS-1_ft.JS-0_ft.IMG-0_ft.MEDIA-0_ft.FONT-0_ft.HTML-0_ft.OTHER-0%22%7D "View CSS assets") | `319.71KiB` | `319.71KiB` | | Not changed  [Fonts](https://app.relative-ci.com/projects/PUROh8FAVkDKmpUrqr4u/jobs/2464-sDdYWs30fnAz8R3NF0xC/assets?ba=%7B%22filters%22%3A%22ft.CSS-0_ft.JS-0_ft.IMG-0_ft.MEDIA-0_ft.FONT-1_ft.HTML-0_ft.OTHER-0%22%7D "View Fonts assets") | `93.55KiB` | `93.55KiB` | | Not changed  [Other](https://app.relative-ci.com/projects/PUROh8FAVkDKmpUrqr4u/jobs/2464-sDdYWs30fnAz8R3NF0xC/assets?ba=%7B%22filters%22%3A%22ft.CSS-0_ft.JS-0_ft.IMG-0_ft.MEDIA-0_ft.FONT-0_ft.HTML-0_ft.OTHER-1%22%7D "View Other assets") | `17.53KiB` | `17.53KiB` | | Not changed  [HTML](https://app.relative-ci.com/projects/PUROh8FAVkDKmpUrqr4u/jobs/2464-sDdYWs30fnAz8R3NF0xC/assets?ba=%7B%22filters%22%3A%22ft.CSS-0_ft.JS-0_ft.IMG-0_ft.MEDIA-0_ft.FONT-0_ft.HTML-1_ft.OTHER-0%22%7D "View HTML assets") | `13.58KiB` | `13.58KiB` |

View job #2464 reportView McFlyPartages:update_mqtt_versio... branch activityView project dashboard

Pierre-Gilles commented 3 months ago

Hey @McFlyPartages, will you do some testing on this or should we close the PR ? Thanks 🙂

McFlyPartages commented 3 months ago

Hy, Sorry I haven't tested it and I don't think I have the time, so you can close the PR. I'll do another if need be