safe-global / safe-config-service / 32381885320
98%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 101
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 97.691%. Remained the same
32381885320

push

github

web-flow 
chore(deps-dev): bump djangorestframework-stubs from 3.17.1 to 3.18.0 (#1668)

* chore(deps-dev): bump djangorestframework-stubs from 3.17.1 to 3.18.0

Bumps [djangorestframework-stubs](https://github.com/typeddjango/djangorestframework-stubs) from 3.17.1 to 3.18.0.
- [Release notes](https://github.com/typeddjango/djangorestframework-stubs/releases)
- [Commits](https://github.com/typeddjango/djangorestframework-stubs/compare/3.17.1...3.18.0)

---
updated-dependencies:
- dependency-name: djangorestframework-stubs
  dependency-version: 3.18.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* fix(migrations): annotate RunPython callables with StateApps

django-stubs 6.1.0, pulled in by the djangorestframework-stubs 3.18.0
bump, narrows the return type of `apps.get_model("app", "Model")` to the
referenced model class. Its mypy plugin deliberately skips that narrowing
for `StateApps`, since a migration's historical models are rebuilt from
migration state and their fields differ from the current definitions.

The `RunPython` callables in `src/chains/migrations` annotated their
`apps` argument as `django.apps.registry.Apps`, so the plugin narrowed
`get_model()` to the current models and `mypy --strict` failed on fields
that no longer exist (and on `chains.WalletNew`, which only ever existed
mid-migration):

    0033_walletnew.py:10: error: Could not match lazy reference with any model
    0033_walletnew.py:13: error: "Wallet" has no attribute "name"
    0023_create_gas_price_model.py:17: error: "Chain" has no attribute "gas_price_oracle_uri"

Annotate those callables with `django.db.migrations.state.StateApps`,
which is what `RunPython` actually passes at runtime and what
django-stubs uses for `RunPython.code`. This matches the annotation the
migration tests in `src/chains/migrations/tests` already use. Runtime
behaviour is unchanged.

The pre-commit license-headers ho... (continued)

1523 of 1559 relevant lines covered (97.69%)

0.98 hits per line

Jobs
ID Job ID Ran Files Coverage
1 32381885320.1 101
97.69
 GitHub Action Run
Source Files on build 32381885320