Stefano Zacchiroli authored 4 years ago and Roberto Di Cosmo committed 4 years ago

Reviewers: rdicosmo

Reviewed By: rdicosmo

Differential Revision: https://forge.softwareheritage.org/D3108

Production should only use aware datetimes.

Closes T1003

 - merge grammars into a single one
 - explain better that SWHIDs are made up of core identifier + qualifiers
 - separate qualifier into context and fragment onex
 - add reference to swh-identify

Closes T2379

Make it possible to generate Content and SkippedContent under different
object types (namely "content" and "skipped_content").

Default to False to keep backward compat.

so one can choose not to generate some of the object types.

Blacklist "origin_visit_status" by default to prevent breaking dependent
packages' tests.

Related to T2367

and user required=True to check that, as it is the preferred way

hat tip to @rdicosmo for noticing

This also adapts the hypothesis strategies, using the plural form
origin_visit_statuses. That plural form is acceptable because in our context,
the statuses are countable.

Related to T2310

- blackify all the python files,
- enable black in pre-commit,
- add a black tox environment.

Hinting at the origin_visit_update model

Related to T2310

A release may have an empty message, for instance those derived from
a Mercurial repository.

So make that field optional to avoid type validation errors.