| mup | PR operator#354 closed: post-release fallback version bump <Created by chipaca> <Merged by jameinel> <https://github.com/canonical/operator/pull/354> | 07:41 |
|---|---|---|
| jam | morning Chipaca, just in time for me to go to lunch :) | 08:45 |
| jam | I put up 2 posts on Discourse, feedback/edits welcome | 08:45 |
| Chipaca | moin moin moin | 08:46 |
| Chipaca | jam: reading | 08:46 |
| Chipaca | jam: the 'reusable components' one, maybe needs tweaking because ops.lib.use is a thing now? | 09:18 |
| jam | Chipaca, we could. Right now the component thing is mostly about what a component looks like. We can add ops.lib.use to it, or we can make that a separate topic | 09:36 |
| Chipaca | k | 09:41 |
| facubatista | Muy buenos días a todos! | 10:56 |
| Chipaca | facubatista: a shame tabulate can't be told to print to a file | 11:40 |
| facubatista | Chipaca, I don't want to print to a file! unless a logger can be handled as a file? | 11:41 |
| Chipaca | facubatista: well, that would be my next 'shame' :-) | 11:41 |
| Chipaca | print(..., file=logger.info) :-) | 11:42 |
| facubatista | don't like that | 11:42 |
| facubatista | you're not pushing bytes/chars to it; it's a complex object which builds messages, etc | 11:42 |
| facubatista | Chipaca, what I *do* miss is: instead of doing `print("foo {} bar {}".format(a, b))`, being able to do `print("foo {} bar {}", a, b)` (and that it doesn't fail if var replacement is wrong) | 11:45 |
| facubatista | printl() | 11:45 |
| facubatista | printf! | 11:45 |
| Chipaca | printf, clearly | 11:45 |
| facubatista | oh | 11:45 |
| Chipaca | facubatista: ? | 11:46 |
| facubatista | now I'm trying to hack it into builtins :p | 11:47 |
| Chipaca | facubatista: it also needs a flogger module that lets you do flogger.debug('the {} is all up the {}', thing, where) | 11:49 |
| facubatista | Chipaca, I want to do that using logger, it's in my project list | 11:50 |
| facubatista | Chipaca, jam, this is the refactor I promised to simplify reviews: https://github.com/canonical/charmcraft/pull/67 | 11:57 |
| mup | PR charmcraft#67: A small refactor creating a "store api" layer between commands and the client <Created by facundobatista> <https://github.com/canonical/charmcraft/pull/67> | 11:57 |
| facubatista | Chipaca, jam, and the refactor about paths in build that I also commented to you: https://github.com/canonical/charmcraft/pull/68 | 12:16 |
| mup | PR charmcraft#68: Totally avoid dealing with paths as strings <Created by facundobatista> <https://github.com/canonical/charmcraft/pull/68> | 12:16 |
| Chipaca | facubatista: so productive 😲 | 12:29 |
| facubatista | :) | 12:30 |
| Chipaca | jam: you coming? | 12:31 |
| jam | yep | 12:31 |
| facubatista | Chipaca, I don't "see" the benefit of using NamedTuple from typing in a not-typed project | 12:57 |
| Chipaca | facubatista: yeah, not that big. I started out with "how to add docstrings" and in 3.6+ you subclass NamedTuple and write a regular docstring there, but in 3.5 you still need to set __doc__ by hand | 12:59 |
| Chipaca | so it's a bit of a wash. Maybe just make a note to switch to NamedTuple once we drop 3.5, such that adding a docstring is nicer? (and … what i should have started with: add a docstring :-) ) | 13:00 |
| facubatista | Chipaca, you mean, adding a docstring to User? | 13:00 |
| Chipaca | facubatista: PosixPath('/tmp/pytest-of-travis/pytest-0/test_validator_requirement_mul0/testfile1') != PosixPath('/tmp/pytest-of-travis/pytest-0/test_validator_requirement_mul0/testfile1') | 13:21 |
| Chipaca | facubatista: a.k.a. 3.5 hates you | 13:22 |
| Chipaca | facubatista: yes i meant adding a docstring to User | 13:22 |
| * Chipaca adds __head__ to all his classes | 13:46 | |
| facubatista | Chipaca, what do you think? https://paste.ubuntu.com/p/j7QBhKSJtf/ | 14:13 |
| Chipaca | facubatista: >3.5 | 14:14 |
| facubatista | Chipaca, why? | 14:15 |
| Chipaca | facubatista: foo:int is a 3.6+ thing | 14:16 |
| facubatista | Chipaca, mmm... so how NamedTuple works in 3.5? | 14:16 |
| * Chipaca points at the docs | 14:16 | |
| Chipaca | facubatista: https://docs.python.org/3.5/library/typing.html#typing.NamedTuple | 14:17 |
| facubatista | Chipaca, and how that helps with the docstring? | 14:18 |
| Chipaca | facubatista: | 14:19 |
| facubatista | right | 14:19 |
| * facubatista opens an issue and forgets about this :p | 14:19 | |
| * Chipaca hugs facubatista | 14:19 | |
| facubatista | Chipaca, https://github.com/canonical/charmcraft/issues/70 | 14:25 |
| facubatista | py3.5 super hates me: https://paste.ubuntu.com/p/qXcZ3wCVP3/ | 14:29 |
| Chipaca | facubatista: You need to upgrade your VESA local bus to a MasterCard local bus. | 14:30 |
| facubatista | Chipaca, I configured the screen to 16 colors and the test still fails | 14:34 |
| * facubatista tries putting it in 320x200 | 14:35 | |
| Chipaca | facubatista: Don't worry; it's been deprecated. The new one is worse. | 14:37 |
| mup | Issue operator#356 opened: Support extra service names <Created by knkski> <https://github.com/canonical/operator/issues/356> | 16:01 |
| * Chipaca EODs | 17:17 | |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!