| opendevreview | sean mooney proposed openstack/watcher-dashboard master: ruff: Add isort config generator and fix per-file-ignores https://review.opendev.org/c/openstack/watcher-dashboard/+/978256 | 13:01 |
|---|---|---|
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: style: replace autopep8 with ruff-format; restrict hacking to H-rules https://review.opendev.org/c/openstack/watcher-dashboard/+/978258 | 13:01 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: test: Restructure test directory with unit/ and local_fixtures/ https://review.opendev.org/c/openstack/watcher-dashboard/+/978257 | 13:01 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: test: remove Selenium and integration tests https://review.opendev.org/c/openstack/watcher-dashboard/+/978304 | 13:01 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: test: strengthen error-path tests and suppress log noise https://review.opendev.org/c/openstack/watcher-dashboard/+/978303 | 13:01 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: test: fix efficacy_specifications fixture to be a list https://review.opendev.org/c/openstack/watcher-dashboard/+/978305 | 13:01 |
| sean-k-mooney | chandankumar: if you base your playwrite serise on https://review.opendev.org/c/openstack/watcher-dashboard/+/978305 | 13:01 |
| sean-k-mooney | that should avoid any conflicts with my ruff rework | 13:02 |
| sean-k-mooney | im going to create 2 other series form it for my onw work 1 adding typing, ceneratlising config parsing and some other techdebg and a second for the service-layer patern refacto which will have its own spec | 13:04 |
| chandankumar | sean-k-mooney: sure | 13:08 |
| sean-k-mooney | thanks | 13:35 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: ruff: Add isort config and fix per-file-ignores https://review.opendev.org/c/openstack/watcher-dashboard/+/978256 | 13:51 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: style: replace autopep8 with ruff-format; restrict hacking to H-rules https://review.opendev.org/c/openstack/watcher-dashboard/+/978258 | 13:51 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: test: Restructure test directory with unit/ and local_fixtures/ https://review.opendev.org/c/openstack/watcher-dashboard/+/978257 | 13:51 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: test: remove Selenium and integration tests https://review.opendev.org/c/openstack/watcher-dashboard/+/978304 | 13:51 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: test: strengthen error-path tests and suppress log noise https://review.opendev.org/c/openstack/watcher-dashboard/+/978303 | 13:51 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: test: fix efficacy_specifications fixture to be a list https://review.opendev.org/c/openstack/watcher-dashboard/+/978305 | 13:51 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: refactor: centralise Django settings access in config.py https://review.opendev.org/c/openstack/watcher-dashboard/+/978301 | 13:51 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: refactor: replace/annotate reflection helpers (A2-1) https://review.opendev.org/c/openstack/watcher-dashboard/+/978302 | 13:51 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: feat(A5-1): add PEP 484 type annotations to api/watcher.py and errors.py https://review.opendev.org/c/openstack/watcher-dashboard/+/978306 | 13:51 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: feat(A5-2): add mypy strict configuration and tox target https://review.opendev.org/c/openstack/watcher-dashboard/+/978307 | 13:51 |
| sean-k-mooney | chandankumar: sorry had a typo in the first commit but the https://review.opendev.org/c/openstack/watcher-dashboard/+/978305 should still be stable assumeing no other issues need to be adressed | 13:52 |
| opendevreview | Victoria Martinez de la Cruz proposed openstack/grian-ui master: Add ruffs isort rules https://review.opendev.org/c/openstack/grian-ui/+/982387 | 14:00 |
| sean-k-mooney | i might have some other cleanups to do on those early patches but ill do that once the ci jobs have had a chance to run | 15:06 |
| chandankumar | ok | 15:11 |
| sean-k-mooney | i moved the doc patch form the first to last commit and i think i missed droping some refences to it in the intermediate commits | 15:12 |
| sean-k-mooney | ill be pushign the last 5 commits shortly. once all 15 are up in the new order ill go through them 1 by one and verify i didnt miss anything imporant and fix up any issues along the way | 15:13 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: test: improve coverage to 100% on api, client, errors, exceptions https://review.opendev.org/c/openstack/watcher-dashboard/+/978309 | 15:53 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: add service-layer functions to api/watcher.py https://review.opendev.org/c/openstack/watcher-dashboard/+/978310 | 15:53 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: migrate content modules to use service layer https://review.opendev.org/c/openstack/watcher-dashboard/+/978311 | 15:53 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: Document UI-layer contribution patterns https://review.opendev.org/c/openstack/watcher-dashboard/+/978239 | 15:53 |
| opendevreview | sean mooney proposed openstack/watcher-dashboard master: migrate skip-action UI to service layer https://review.opendev.org/c/openstack/watcher-dashboard/+/982674 | 15:53 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!