Page MenuHomeSoftware Heritage

auth/utils: Fix circular import triggered by sphinx autodoc
ClosedPublic

Authored by anlambert on Aug 23 2022, 12:01 PM.

Details

Summary

Fix "cannot import name 'ADD_FORGE_MODERATOR_PERMISSION' from
'swh.web.auth.utils'" when autodoc processes the admin_views
module from swh.web.add_forge_now.

https://jenkins.softwareheritage.org/job/DDOC/job/dev/2156/artifact/swh-docs/docs/errors.log

Also remove no longer needed swh/web/common folder.

Diff Detail

Repository
rDWAPPS Web applications
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Build is green

Patch application report for D8290 (id=29938)

Rebasing onto 1975f2b38b...

Current branch diff-target is up to date.
Changes applied before test
commit cb2f35021f60dc816c7fa326100f6c20b1ab5192
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Tue Aug 23 11:59:00 2022 +0200

    auth/utils: Fix circular import triggered by sphinx autodoc
    
    Fix "cannot import name 'ADD_FORGE_MODERATOR_PERMISSION' from
    'swh.web.auth.utils'" when autodoc processes the admin_views
    module from swh.web.add_forge_now.

commit 57c6590017b7d2217782a58e18e312dd11dc8f13
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Mon Aug 22 13:17:29 2022 +0200

    Remove no longer needed swh/web/common folder

See https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/1994/ for more details.

This revision is now accepted and ready to land.Aug 23 2022, 1:04 PM