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

Move auto updater to trigger #778

Merged
merged 5 commits into from
Jul 21, 2024
Merged

Move auto updater to trigger #778

merged 5 commits into from
Jul 21, 2024

Conversation

davidnewhall
Copy link
Contributor

The auto updater was kinda stabbed in poorly. This removes that poor code and enriches it by moving it to normal action trigger with a timer. Now it shares logging and a loop with other triggers.

Bunch of other random cleanup.

@davidnewhall davidnewhall merged commit 12a7c92 into main Jul 21, 2024
17 checks passed
@davidnewhall davidnewhall deleted the dn2_cleaning branch July 28, 2024 01:30
truecharts-admin referenced this pull request in truecharts/public Jul 28, 2024
…8 by renovate (#24477)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [golift/notifiarr](https://togithub.com/Notifiarr/notifiarr) | patch |
`0.8.0` -> `0.8.1` |

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>Notifiarr/notifiarr (golift/notifiarr)</summary>

###
[`v0.8.1`](https://togithub.com/Notifiarr/notifiarr/releases/tag/v0.8.1)

[Compare
Source](https://togithub.com/Notifiarr/notifiarr/compare/v0.8.0...v0.8.1)

#### Changes

- Moves Windows auto updater to a trigger. It now works correctly in all
scenarios.
-   Cleans up backup executables on Windows.
-   Makes X11 GUI support work correctly in Linux.
-   Adds an integrity check so we can identify custom clients.
-   Service check is now disabled when no services to check exist.
-   New trigger interval to send an up-check to the website.
-   Removes home-brew builds.
- Nvidia-SMI path can now be removed in UI. App will correctly use PATH
to find it.
-   Client was, and no longer, sends mysql auth info to the website.
- Fixes a bug in Arch aur package maker than was putting files in /share
instead of /usr/share.
-   Adds `auto-start` checkbox to GUI menu.

#### Auto Generated Notes

- Move auto updater to trigger by
[@&#8203;davidnewhall](https://togithub.com/davidnewhall) in
[https://github.com/Notifiarr/notifiarr/pull/778](https://togithub.com/Notifiarr/notifiarr/pull/778)
- Move instance tests to their own package by
[@&#8203;davidnewhall](https://togithub.com/davidnewhall) in
[https://github.com/Notifiarr/notifiarr/pull/779](https://togithub.com/Notifiarr/notifiarr/pull/779)
- Code cleaning with linters by
[@&#8203;davidnewhall](https://togithub.com/davidnewhall) in
[https://github.com/Notifiarr/notifiarr/pull/780](https://togithub.com/Notifiarr/notifiarr/pull/780)
- More Linux/FreeBSD GUI support. by
[@&#8203;davidnewhall](https://togithub.com/davidnewhall) in
[https://github.com/Notifiarr/notifiarr/pull/781](https://togithub.com/Notifiarr/notifiarr/pull/781)
- fix(deps): update module modernc.org/sqlite to v1.31.1 by
[@&#8203;renovate](https://togithub.com/renovate) in
[https://github.com/Notifiarr/notifiarr/pull/782](https://togithub.com/Notifiarr/notifiarr/pull/782)
- fix(deps): update golift.io/rotatorr digest to
[`cb73b9c`](https://togithub.com/Notifiarr/notifiarr/commit/cb73b9c) by
[@&#8203;renovate](https://togithub.com/renovate) in
[https://github.com/Notifiarr/notifiarr/pull/783](https://togithub.com/Notifiarr/notifiarr/pull/783)
- add more linux desktop features by
[@&#8203;davidnewhall](https://togithub.com/davidnewhall) in
[https://github.com/Notifiarr/notifiarr/pull/785](https://togithub.com/Notifiarr/notifiarr/pull/785)
- add version to step names by
[@&#8203;davidnewhall](https://togithub.com/davidnewhall) in
[https://github.com/Notifiarr/notifiarr/pull/786](https://togithub.com/Notifiarr/notifiarr/pull/786)
- fix upcheck by
[@&#8203;davidnewhall](https://togithub.com/davidnewhall) in
[https://github.com/Notifiarr/notifiarr/pull/787](https://togithub.com/Notifiarr/notifiarr/pull/787)
- add integrity check to system page by
[@&#8203;davidnewhall](https://togithub.com/davidnewhall) in
[https://github.com/Notifiarr/notifiarr/pull/788](https://togithub.com/Notifiarr/notifiarr/pull/788)
- add backup cleanup codez by
[@&#8203;davidnewhall](https://togithub.com/davidnewhall) in
[https://github.com/Notifiarr/notifiarr/pull/789](https://togithub.com/Notifiarr/notifiarr/pull/789)

**Full Changelog**:
Notifiarr/notifiarr@v0.8.0...v0.8.1

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **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://togithub.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC44LjMiLCJ1cGRhdGVkSW5WZXIiOiIzOC44LjMiLCJ0YXJnZXRCcmFuY2giOiJtYXN0ZXIiLCJsYWJlbHMiOlsiYXV0b21lcmdlIiwidXBkYXRlL2RvY2tlci9nZW5lcmFsL25vbi1tYWpvciJdfQ==-->
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.

1 participant