[04:52] Issue operator#317 opened: use charm state for Juju 2.8 [08:29] morning all [08:33] morning Chipaca [11:04] Muy buenos días a todos! [11:12] a wild facundo appears! [11:14] * facubatista puts some clothes on [11:25] better [11:25] facubatista: hola :) [11:25] hola :) [11:25] i've just realised i spent all morning messing around with versioning :-( [11:25] Chipaca, and this is the 3.1.2nd time it happes to you! [11:26] *happens [11:26] well, i did do a bit of reviewing and such before diving down this rabbithole [11:26] Chipaca, but at least you have a conclusion? [11:26] facubatista: I have something I like, I just need to check one more thing [11:28] hm, the files github gives us at https://github.com/canonical/operator/releases is already reasonably useless :-/ do we care about them? [11:29] Chipaca, for `fades`, the person doing the debian packaging takes those [11:30] Chipaca, anyway, I don't care about them, not even to take the action of removing them or something [11:30] heh [11:31] Chipaca, re MANIFEST.in... "LICENSE": don't we need to package it? why do we have it in the first place? shall we remove it from the project? [11:32] facubatista: in the wheel, it's already there [11:32] facubatista: in the source dist, we reference it by name, isn't that enough? [11:33] i'm asking :) [11:33] Chipaca, who put it in the wheel? [11:36] facubatista: wheel :) [11:37] facubatista: FWIW I don't object to it being there [11:37] Chipaca, even if we don't specify to put it in the tarball? [11:37] facubatista: correct [11:37] wow [11:37] facubatista: you can try it yourself :) [11:37] nah, I trust you [11:37] facubatista: 'python setup.py bdist_wheel' [11:37] shaaame [11:37] oh dang lunch [11:38] * Chipaca rushes [12:13] so, it *looks* like github doesn't auto-add assets if you add them manually, so that might be a good way out [12:15] Chipaca, what we do about LICENSE file, then? [12:16] facubatista: give me a bit [12:16] just digging into that :) [12:16] bah. i should probably shelve the version stuff first :-D [12:16] ack, no rush [12:21] facubatista: so one approach is to add a setup.cfg with [metadata] license_file = LICENSE.txt [12:21] license_files* [12:26] Chipaca, jam, now I pushed the files to where they belong, so I won't confuse again my directories: https://github.com/facundobatista/blog/tree/master/charm [12:30] i don't have a stick big enough to shake at how much i dislike the under-documentedness of setup.py and .cfg [13:08] Chipaca, simple enough :) [13:10] grmbl [13:10] (tm) [13:17] Chipaca, see, I created a clean venv, then `pip install -r requirements-dev.txt`, and I get: [13:17] Installing collected packages: pycodestyle, toml, autopep8, mccabe, zipp, importlib-metadata, pyflakes, flake8, PyYAML, ops [13:17] I don't like that "ops" at the end [13:18] oh, mmm [13:18] $ cat env/lib/python3.6/site-packages/ops.egg-link [13:18] /home/facundo/canonical/operator [13:18] probably not harmful, though [13:19] yep, it's done a 'develop' [14:03] * Chipaca links to https://xkcd.com/1987/ on the README.md [14:05] Chipaca, :) [14:06] danger of using 'sudo' to install python packages, indeed [14:08] PR operator#318 opened: Simplified the CharmBase init signature [14:09] never ever do "sudo pip" [14:09] facubatista, do "curl | sudo bash" right? [14:10] ja [14:18] oh, Trello is down :/ [14:57] facubatista: if trello is down, are you even working [15:00] Chipaca, I'm not doing anything but pressing F5 every couple of minutes, staring at the empty browser page [15:01] facubatista: let's have our 1:1 now then :) [15:02] je [15:03] Chipaca, bah, I'm ok for the 1:1 now if you prefer it [15:06] facubatista: let's do it [15:06] ok, 1' [15:08] Chipaca, omw [16:43] soft EOD from me. brain = ded [16:43] migt bbl if it reactivates [17:10] Chipaca, should we distribute ops docs inside its package? === facundo__ is now known as facubatista [21:04] * facundo__ eods [22:24] * Chipaca eods