synapse/changelog.d
Amber Brown 129ffd7b88
Merge pull request #3498 from OlegGirko/fix_attrs_syntax
* Use more portable syntax using attrs package.

Newer syntax

    attr.ib(factory=dict)

is just a syntactic sugar for

    attr.ib(default=attr.Factory(dict))

It was introduced in newest version of attrs package (18.1.0)
and doesn't work with older versions.

We should either require minimum version of attrs to be 18.1.0,
or use older (slightly more verbose) syntax.
Requiring newest version is not a good solution because
Linux distributions may have older version of attrs (17.4.0 in Fedora 28),
and requiring to build (and package)
newer version just to use newer syntactic sugar in only one test
is just too much.
It's much better to fix that test to use older syntax.

Signed-off-by: Oleg Girko <ol@infoserver.lv>
2018-07-11 04:22:46 +10:00
..
.gitignore Correct attrs package name in requirements (#3492) 2018-07-07 10:46:59 +10:00
3463.misc Add an isort configuration (#3463) 2018-07-09 16:05:21 +10:00
3464.misc changelog 2018-07-09 16:09:37 +10:00
3496.feature Attempt to include db threads in cpu usage stats (#3496) 2018-07-10 16:12:36 +01:00
3497.feature changelog 2018-07-09 18:16:52 +01:00
3498.misc changelog entry 2018-07-11 03:27:03 +10:00
3501.misc Refactor logcontext resource usage tracking (#3501) 2018-07-10 13:56:07 +01:00