search

MatterMiners / tardis

Transparent Adaptive Resource Dynamic Integration System
https://cobald-tardis.readthedocs.io
MIT License
15 stars 20 forks source link

Fix broken yaml tags in the documentation #309

Closed giffels closed 11 months ago

giffels commented 11 months ago

Since we migrated to cobald.config.yaml_constructors, the old-style yaml tags are not working anymore. This pull request fixes broken old-stlye yaml tags in the documentation. Thanks to @harrypuuter for reporting this issue.

codecov-commenter commented 11 months ago

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.04% :warning:

Comparison is base (48daacb) 98.90% compared to head (7fc6ae5) 98.86%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #309 +/- ## ========================================== - Coverage 98.90% 98.86% -0.04% ========================================== Files 55 55 Lines 2277 2210 -67 ========================================== - Hits 2252 2185 -67 Misses 25 25 ``` | [Files Changed](https://app.codecov.io/gh/MatterMiners/tardis/pull/309?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MatterMiners) | Coverage Δ | | |---|---|---| | [tardis/adapters/sites/cloudstack.py](https://app.codecov.io/gh/MatterMiners/tardis/pull/309?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MatterMiners#diff-dGFyZGlzL2FkYXB0ZXJzL3NpdGVzL2Nsb3Vkc3RhY2sucHk=) | `100.00% <ø> (ø)` | | | [tardis/resources/drone.py](https://app.codecov.io/gh/MatterMiners/tardis/pull/309?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MatterMiners#diff-dGFyZGlzL3Jlc291cmNlcy9kcm9uZS5weQ==) | `100.00% <ø> (ø)` | | | [tardis/adapters/sites/fakesite.py](https://app.codecov.io/gh/MatterMiners/tardis/pull/309?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MatterMiners#diff-dGFyZGlzL2FkYXB0ZXJzL3NpdGVzL2Zha2VzaXRlLnB5) | `100.00% <100.00%> (ø)` | | | [tardis/adapters/sites/htcondor.py](https://app.codecov.io/gh/MatterMiners/tardis/pull/309?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MatterMiners#diff-dGFyZGlzL2FkYXB0ZXJzL3NpdGVzL2h0Y29uZG9yLnB5) | `100.00% <100.00%> (ø)` | | | [tardis/adapters/sites/kubernetes.py](https://app.codecov.io/gh/MatterMiners/tardis/pull/309?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MatterMiners#diff-dGFyZGlzL2FkYXB0ZXJzL3NpdGVzL2t1YmVybmV0ZXMucHk=) | `100.00% <100.00%> (ø)` | | | [tardis/adapters/sites/moab.py](https://app.codecov.io/gh/MatterMiners/tardis/pull/309?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MatterMiners#diff-dGFyZGlzL2FkYXB0ZXJzL3NpdGVzL21vYWIucHk=) | `100.00% <100.00%> (ø)` | | | [tardis/adapters/sites/openstack.py](https://app.codecov.io/gh/MatterMiners/tardis/pull/309?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MatterMiners#diff-dGFyZGlzL2FkYXB0ZXJzL3NpdGVzL29wZW5zdGFjay5weQ==) | `100.00% <100.00%> (ø)` | | | [tardis/adapters/sites/slurm.py](https://app.codecov.io/gh/MatterMiners/tardis/pull/309?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MatterMiners#diff-dGFyZGlzL2FkYXB0ZXJzL3NpdGVzL3NsdXJtLnB5) | `100.00% <100.00%> (ø)` | | | [tardis/interfaces/siteadapter.py](https://app.codecov.io/gh/MatterMiners/tardis/pull/309?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MatterMiners#diff-dGFyZGlzL2ludGVyZmFjZXMvc2l0ZWFkYXB0ZXIucHk=) | `100.00% <100.00%> (ø)` | |

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

maxfischer2781 commented 11 months ago

Looks like there is a bit more in here than just the YAML stuff. Is this intentional?

giffels commented 11 months ago

Looks like there is a bit more in here than just the YAML stuff. Is this intentional?

Thanks for notifing. Nope, already cleaned it again.