Tony Meyer

Add black as style guide choice

1

It would be fantastic if [black](https://github.com/ambv/black) was available as another style guide choice in the reformat code functionality. Black is rapidly gaining popularity within the Python world, so seems like...

Persist feeds sidebar open/close state

6

The first thing I do every time I open NNW (if it has been terminated) is hide the feeds sidebar. It would be great if this choice was saved so...

Minor adjustments to the way that we record changes: * Add more structure to [CHANGES](CHANGES.md): group changes by the same types of groups as in our conventional commit messages (but...

Simplify testing code that checks the Juju version

4

Charms, such as [mysql-k8s-operator](https://github.com/canonical/mysql-k8s-operator/blob/128d32fd720f65b18a69650bb827dddf64a2b940/lib/charms/mysql/v0/mysql.py#L585), have code like: ```python if ops.jujuversion.JujuVersion.from_environ().has_secrets: ``` in order to determine whether Juju features are available (and behave differently as a result). However, the default version...

Extend ops.log to capture warnings

1

Although charms can simply use the `logging` module for everything, warnings generated with the `warnings` module have some nice filtering functionality and are generally the more Pythonic way to indicate...

remote unit data is inaccessible in relation-departed

2

The [docs](https://juju.is/docs/sdk/relation-name-relation-departed-event#heading--relation-data) say: > A unit’s relation settings persist beyond its own departure from the relation [...] > During a relation-departed hook, relation settings can still be read (with relation-get)...

Add retry options to Container methods

2

If there's a temporary (very short-lived) issue communicating with Pebble, then we generally recommend that charms retry a small number of times (with appropriate pauses between attempts) rather than `defer()`,...

Run deferred Juju events on a fresh charm instance

2

The main benefit here would be that it would be simpler to reason about (Juju) event behaviour, because it would be consistent both when an event was deferred and when...

Add a `__str__` to `ops.testing.ActionFailed` so that an uncaught `event.fail()` in tests produces a more informative traceback. Currently, `event.fail()` and `event.fail(message)` will look somewhat like: ```python Traceback (most recent call...

Now that the upstream fixes are merged, these should pass again.