Looking into the feasibility of making a change to the auth code.
Before changing anything around, I wanted to get the coverage up.
Overview:
Blacken code.
Simplify code.
Fill some test holes.
Full details in the individual commit messages (easier to review changes one by one).
As always with auth code changes (even ones like this which shouldn't change any functionality), please extensively test this (note you can use exvcylc17 for this)!
Plz squash merge.
Check List
[x] I have read CONTRIBUTING.md and added my name as a Code Contributor.
[x] Contains logically grouped changes (else tidy your branch by rebase).
[x] Does not contain off-topic changes (use other PRs for other changes).
[x] Applied any dependency changes to both setup.cfg (and conda-environment.yml if present).
[x] Tests are included (or explain why tests are not needed).
[x] CHANGES.md entry included if this is a change that can affect users
[x] Cylc-Doc pull request opened if required at cylc/cylc-doc/pull/XXXX.
[x] If this is a bug fix, PR should be raised against the relevant ?.?.x branch.
(low priority)
Looking into the feasibility of making a change to the auth code.
Before changing anything around, I wanted to get the coverage up.
Overview:
Full details in the individual commit messages (easier to review changes one by one).
As always with auth code changes (even ones like this which shouldn't change any functionality), please extensively test this (note you can use exvcylc17 for this)!
Plz squash merge.
Check List
CONTRIBUTING.md
and added my name as a Code Contributor.setup.cfg
(andconda-environment.yml
if present).CHANGES.md
entry included if this is a change that can affect users?.?.x
branch.