[08:09] morning all! [08:21] morning [09:57] bthomas: I pushed a bunch of changes to #417, mostly focused on fixing the rst but also some other misc [09:57] PR #417: Added a few doc strings to charm module [09:57] shout if you don't like any of it :) [09:57] Chipaca: Will have a look. Thank you. Trust your judgement :) [09:57] *I* don't, but thanks [09:58] bthomas: I suggest you install docs/requirements.txt and run ./build_docs, and look at docs/_build/html/index.html in your browser [09:58] ok [09:58] that's what i did, lacking aforementioned trust in my own judgement 🙂 [09:58] he he [09:59] sphinx seems a lot happier now [10:16] workers are sawing up the road outside my window [10:16] and it's almost lunchtime [10:16] so, runch [10:16] * Chipaca ⇝ runch [10:39] ¡Muy buenos días a todos! [10:40] नमस्ते facubatista [10:40] Chipaca: read through the built docs. The formatting looks clean. Thank you. There are things about the content I can still work on though, and will do so as my understanding grows and have some time. For instance in ops.Charm.CharmBase and and a few other classes repeat the information/example of EventSource/EventBase multiple times. It may be more illuminating to provide more contextual examples. But I guess this is a job for me, for [10:40] another day. [10:40] hola bthomas [11:27] * bthomas -> munch, munch [11:34] facubatista: buen día señor! [11:50] hola Chipaca :) [13:56] * facubatista brb, errands [15:24] WELL [15:24] AssertionError: Please fix files as suggested by pydocstyle (461 issues): [15:24] * facubatista is adding docstring check to ops [15:25] (note that count is ignoring test cases and special methods) [15:27] facubatista: ooh, i was looking at that earlier today too [15:27] PR operator#424 opened: Added docstrings checks [15:28] facubatista: pydocstyle --convention=google --add-ignore=D105,D107 ops [15:28] facubatista: still a long way to go [15:28] Nice [15:29] facubatista: i don't know what the equivalent of --convention=google is but please do that also :) [15:32] much better now! [15:32] AssertionError: Please fix files as suggested by pydocstyle (102 issues): [15:32] facubatista: stand up? [15:33] oops [17:03] this is EOD for me peeps [17:03] 👋 [17:16] * bthomas dito [17:36] PR operator#425 opened: Fixed several small details in lot of docstrings [17:36] Easy review! ↑ [17:44] facubatista: can we block that until #417 is merged please [17:44] PR #417: Added a few doc strings to charm module [17:47] Uh, I forgot to talk about 417 today in the standup, it's almost 3 weeks now, do we see a light at the end of the tunnel for it? [17:58] facubatista: I will also try and remember next standup. There are a lot of details that can be refined still. As long as Chipaca, jam and jldev are ok to merge we can merge it, and then make incremental changes as time passes. In response to some questions and comments by jam I suggested changes asking for his approval. If he accepts those suggestions I can make the changes before merge or we can do it later. [18:23] bthomas, landing at 90% right and improve the remaining 10% is fine for me, I prefer that to waiting for it to be 101% right [18:23] (it's similar to release early, release often) [18:23] facubatista: agreed [20:42] reviews appreciated, thanks! Improvements per ux review - https://github.com/canonical/charmcraft/pull/171 [20:42] PR charmcraft#171: Improvements per ux review