Christian Loos

@lsegal can you please merge this PR!

Please note that `/etc/apt/trusted.gpg.d/` is the wrong location for the keys. Also `/usr/share/keyrings`, which is mentioned in other places is wrong, because this directory should only be used by keys...

@bcoca I think `/usr/share/keyrings` should not be supported, because as Debian says in the Wiki: >If future updates to the key will be managed by an apt/dpkg package as recommended...

Here a link to the documentation where GitHub recommends using major version tags: https://docs.github.com/en/actions/creating-actions/about-custom-actions#using-tags-for-release-management

A better restart policy would be `unless-stopped` instead of `always`: https://docs.docker.com/config/containers/start-containers-automatically/#use-a-restart-policy `unless-stopped` prevents containers from starting on Docker daemon restart, if they where previously stopped manually.

@tigattack I can confirm that running step 1 twice helps in the situation you described. But as noted above, I see the most important problem with the change in step...

I did some testing today. The problem is that in https://github.com/ansible-collections/community.windows/blob/a45cc22674a1f4e16702622ab3b5d6b41c920396/plugins/modules/win_dhcp_lease.ps1#L342 the reservation is always updated. And `Set-DhcpServerv4Reservation` always update the reservation and set the `AddressState` to `InactiveReservation`. And as...

@atus42 you can also use the queue id instead of the queue name to workaround your problem. This also have the benefit that you don't have to change the alias...

@atus42 I think new JavaScript shouldn't be in-lined (share/static/js/forms.js would be a better place for your JavaScript). This makes it possible, once all in-line JavaScript is gone, to use HTTP...

I like @ruz idea. You can see the decoded and encoded version without switching a config option.