Page MenuHomeSoftware Heritage

Add annotations to the main function in __init__.py
ClosedPublic

Authored by VickyMerzOwn on Apr 15 2022, 11:16 AM.

Diff Detail

Repository
rDCIDX Metadata indexer
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 28550
Build 44638: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 44637: arc lint + arc unit

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes

Hello,

Thanks for your contribution.

@VickyMerzOwn you need to sign the L3 document (linked in this very diff) so we can review your diff though.

Cheers,

lgtm.

Please, use imperative form in your commit message though.
Here that'd be: "Add annotations to the main function in __init__.py".

More information on our docs [1].

[1] https://docs.softwareheritage.org/devel/contributing/git-style-guide.html#link-commits-to-tasks

Cheers,

VickyMerzOwn retitled this revision from Added annotations to the main function in __init__.py to Adds annotations to the main function in __init__.py.Apr 16 2022, 7:55 AM
VickyMerzOwn retitled this revision from Adds annotations to the main function in __init__.py to Add annotations to the main function in __init__.py.

Add annotations to the functions in base.py

Build is green

Patch application report for D7587 (id=27497)

Rebasing onto 4fe8cacf21...

Current branch diff-target is up to date.
Changes applied before test
commit 906ab2ac21aa602c3e48e3fbbecf09487c535610
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Fri Apr 15 14:32:05 2022 +0530

    Added annotations to the main function in __init__.py

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

@ardumont Is this good enough to be merged? If so, there are other annotations that need to be added and I'll create those diffs as well.

Build has FAILED

Patch application report for D7587 (id=27498)

Rebasing onto 4fe8cacf21...

Current branch diff-target is up to date.
Changes applied before test
commit 07bfc71539d4cb7fa3a3910eb2bdaa2c34b193cb
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 14:55:02 2022 +0530

    Add annotations to base.py

commit 906ab2ac21aa602c3e48e3fbbecf09487c535610
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Fri Apr 15 14:32:05 2022 +0530

    Added annotations to the main function in __init__.py

Link to build: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/203/
See console output for more information: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/203/console

Build has FAILED

Patch application report for D7587 (id=27499)

Rebasing onto 4fe8cacf21...

Current branch diff-target is up to date.
Changes applied before test
commit e535236000516516ba64de1078c5e4c7ca72d7d8
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:06:56 2022 +0530

    Add annotations to base.py

commit 07bfc71539d4cb7fa3a3910eb2bdaa2c34b193cb
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 14:55:02 2022 +0530

    Add annotations to base.py

commit 906ab2ac21aa602c3e48e3fbbecf09487c535610
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Fri Apr 15 14:32:05 2022 +0530

    Added annotations to the main function in __init__.py

Link to build: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/204/
See console output for more information: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/204/console

Build is green

Patch application report for D7587 (id=27500)

Rebasing onto 4fe8cacf21...

Current branch diff-target is up to date.
Changes applied before test
commit 74ad0bc730c908132c2a1f514e79ecd5b475af8b
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:23:38 2022 +0530

    Add annotations to base.py

commit e535236000516516ba64de1078c5e4c7ca72d7d8
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:06:56 2022 +0530

    Add annotations to base.py

commit 07bfc71539d4cb7fa3a3910eb2bdaa2c34b193cb
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 14:55:02 2022 +0530

    Add annotations to base.py

commit 906ab2ac21aa602c3e48e3fbbecf09487c535610
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Fri Apr 15 14:32:05 2022 +0530

    Added annotations to the main function in __init__.py

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

Thanks.

Please use Optional[...] instead of Union[..., None].

And try to make signatures more specific:

Build has FAILED

Patch application report for D7587 (id=27501)

Rebasing onto 4fe8cacf21...

Current branch diff-target is up to date.
Changes applied before test
commit 334d50fa304f05e7286fe07ddc174825d2000c43
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 20:15:59 2022 +0530

    Add annotations to cff.py

commit 74ad0bc730c908132c2a1f514e79ecd5b475af8b
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:23:38 2022 +0530

    Add annotations to base.py

commit e535236000516516ba64de1078c5e4c7ca72d7d8
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:06:56 2022 +0530

    Add annotations to base.py

commit 07bfc71539d4cb7fa3a3910eb2bdaa2c34b193cb
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 14:55:02 2022 +0530

    Add annotations to base.py

commit 906ab2ac21aa602c3e48e3fbbecf09487c535610
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Fri Apr 15 14:32:05 2022 +0530

    Added annotations to the main function in __init__.py

Link to build: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/206/
See console output for more information: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/206/console

Please use Optional[...] instead of Union[..., None].

I have to use `Optional[Union[...,...]]`, right?

And try to make signatures more specific:

Okay, I'll do this as well.

Build is green

Patch application report for D7587 (id=27502)

Rebasing onto 4fe8cacf21...

Current branch diff-target is up to date.
Changes applied before test
commit 05f68f33fe7894d2791b9cb8260f5a33706f53a1
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 21:49:18 2022 +0530

    Add annotations to cff.py

commit 334d50fa304f05e7286fe07ddc174825d2000c43
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 20:15:59 2022 +0530

    Add annotations to cff.py

commit 74ad0bc730c908132c2a1f514e79ecd5b475af8b
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:23:38 2022 +0530

    Add annotations to base.py

commit e535236000516516ba64de1078c5e4c7ca72d7d8
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:06:56 2022 +0530

    Add annotations to base.py

commit 07bfc71539d4cb7fa3a3910eb2bdaa2c34b193cb
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 14:55:02 2022 +0530

    Add annotations to base.py

commit 906ab2ac21aa602c3e48e3fbbecf09487c535610
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Fri Apr 15 14:32:05 2022 +0530

    Added annotations to the main function in __init__.py

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

  • Add some more annotations to base.py
  • Add some more annotations to cff.py

Build has FAILED

Patch application report for D7587 (id=27503)

Rebasing onto 4fe8cacf21...

Current branch diff-target is up to date.
Changes applied before test
commit 17072eb9a583fc609fa922bb8bd69532ea815d63
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 00:08:38 2022 +0530

    Add some more annotations to cff.py

commit f0b3401c1cee303fdabc9cc498c1b414d83f6e5e
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 23:53:27 2022 +0530

    Add some more annotations to base.py

commit 05f68f33fe7894d2791b9cb8260f5a33706f53a1
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 21:49:18 2022 +0530

    Add annotations to cff.py

commit 334d50fa304f05e7286fe07ddc174825d2000c43
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 20:15:59 2022 +0530

    Add annotations to cff.py

commit 74ad0bc730c908132c2a1f514e79ecd5b475af8b
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:23:38 2022 +0530

    Add annotations to base.py

commit e535236000516516ba64de1078c5e4c7ca72d7d8
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:06:56 2022 +0530

    Add annotations to base.py

commit 07bfc71539d4cb7fa3a3910eb2bdaa2c34b193cb
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 14:55:02 2022 +0530

    Add annotations to base.py

commit 906ab2ac21aa602c3e48e3fbbecf09487c535610
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Fri Apr 15 14:32:05 2022 +0530

    Added annotations to the main function in __init__.py

Link to build: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/208/
See console output for more information: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/208/console

Add annotations to __init__.py, base.py and cff.py

Build has FAILED

Patch application report for D7587 (id=27504)

Rebasing onto 4fe8cacf21...

Current branch diff-target is up to date.
Changes applied before test
commit 17072eb9a583fc609fa922bb8bd69532ea815d63
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 00:08:38 2022 +0530

    Add some more annotations to cff.py

commit f0b3401c1cee303fdabc9cc498c1b414d83f6e5e
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 23:53:27 2022 +0530

    Add some more annotations to base.py

commit 05f68f33fe7894d2791b9cb8260f5a33706f53a1
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 21:49:18 2022 +0530

    Add annotations to cff.py

commit 334d50fa304f05e7286fe07ddc174825d2000c43
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 20:15:59 2022 +0530

    Add annotations to cff.py

commit 74ad0bc730c908132c2a1f514e79ecd5b475af8b
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:23:38 2022 +0530

    Add annotations to base.py

commit e535236000516516ba64de1078c5e4c7ca72d7d8
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:06:56 2022 +0530

    Add annotations to base.py

commit 07bfc71539d4cb7fa3a3910eb2bdaa2c34b193cb
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 14:55:02 2022 +0530

    Add annotations to base.py

commit 906ab2ac21aa602c3e48e3fbbecf09487c535610
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Fri Apr 15 14:32:05 2022 +0530

    Added annotations to the main function in __init__.py

Link to build: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/209/
See console output for more information: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/209/console

  • Add some more annotations to base.py

Build is green

Patch application report for D7587 (id=27505)

Rebasing onto 4fe8cacf21...

Current branch diff-target is up to date.
Changes applied before test
commit dd6ca459b588b1b891d41cac57cdd8d0d2cab78f
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 02:37:00 2022 +0530

    Add some more annotations to base.py

commit 17072eb9a583fc609fa922bb8bd69532ea815d63
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 00:08:38 2022 +0530

    Add some more annotations to cff.py

commit f0b3401c1cee303fdabc9cc498c1b414d83f6e5e
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 23:53:27 2022 +0530

    Add some more annotations to base.py

commit 05f68f33fe7894d2791b9cb8260f5a33706f53a1
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 21:49:18 2022 +0530

    Add annotations to cff.py

commit 334d50fa304f05e7286fe07ddc174825d2000c43
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 20:15:59 2022 +0530

    Add annotations to cff.py

commit 74ad0bc730c908132c2a1f514e79ecd5b475af8b
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:23:38 2022 +0530

    Add annotations to base.py

commit e535236000516516ba64de1078c5e4c7ca72d7d8
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:06:56 2022 +0530

    Add annotations to base.py

commit 07bfc71539d4cb7fa3a3910eb2bdaa2c34b193cb
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 14:55:02 2022 +0530

    Add annotations to base.py

commit 906ab2ac21aa602c3e48e3fbbecf09487c535610
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Fri Apr 15 14:32:05 2022 +0530

    Added annotations to the main function in __init__.py

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

  • Add annotations to codemeta.py

Build is green

Patch application report for D7587 (id=27506)

Rebasing onto 4fe8cacf21...

Current branch diff-target is up to date.
Changes applied before test
commit a7198bea6acfaeda8ff18be6ffe1e869966924e4
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 02:57:43 2022 +0530

    Add annotations to codemeta.py

commit dd6ca459b588b1b891d41cac57cdd8d0d2cab78f
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 02:37:00 2022 +0530

    Add some more annotations to base.py

commit 17072eb9a583fc609fa922bb8bd69532ea815d63
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 00:08:38 2022 +0530

    Add some more annotations to cff.py

commit f0b3401c1cee303fdabc9cc498c1b414d83f6e5e
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 23:53:27 2022 +0530

    Add some more annotations to base.py

commit 05f68f33fe7894d2791b9cb8260f5a33706f53a1
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 21:49:18 2022 +0530

    Add annotations to cff.py

commit 334d50fa304f05e7286fe07ddc174825d2000c43
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 20:15:59 2022 +0530

    Add annotations to cff.py

commit 74ad0bc730c908132c2a1f514e79ecd5b475af8b
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:23:38 2022 +0530

    Add annotations to base.py

commit e535236000516516ba64de1078c5e4c7ca72d7d8
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:06:56 2022 +0530

    Add annotations to base.py

commit 07bfc71539d4cb7fa3a3910eb2bdaa2c34b193cb
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 14:55:02 2022 +0530

    Add annotations to base.py

commit 906ab2ac21aa602c3e48e3fbbecf09487c535610
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Fri Apr 15 14:32:05 2022 +0530

    Added annotations to the main function in __init__.py

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

Build is green

Patch application report for D7587 (id=27507)

Rebasing onto 4fe8cacf21...

Current branch diff-target is up to date.
Changes applied before test
commit f885ab3b45be36ac83511153eb0673c1d4fda727
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 03:44:22 2022 +0530

    Add annotations to maven.py

commit a7198bea6acfaeda8ff18be6ffe1e869966924e4
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 02:57:43 2022 +0530

    Add annotations to codemeta.py

commit dd6ca459b588b1b891d41cac57cdd8d0d2cab78f
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 02:37:00 2022 +0530

    Add some more annotations to base.py

commit 17072eb9a583fc609fa922bb8bd69532ea815d63
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 00:08:38 2022 +0530

    Add some more annotations to cff.py

commit f0b3401c1cee303fdabc9cc498c1b414d83f6e5e
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 23:53:27 2022 +0530

    Add some more annotations to base.py

commit 05f68f33fe7894d2791b9cb8260f5a33706f53a1
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 21:49:18 2022 +0530

    Add annotations to cff.py

commit 334d50fa304f05e7286fe07ddc174825d2000c43
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 20:15:59 2022 +0530

    Add annotations to cff.py

commit 74ad0bc730c908132c2a1f514e79ecd5b475af8b
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:23:38 2022 +0530

    Add annotations to base.py

commit e535236000516516ba64de1078c5e4c7ca72d7d8
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:06:56 2022 +0530

    Add annotations to base.py

commit 07bfc71539d4cb7fa3a3910eb2bdaa2c34b193cb
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 14:55:02 2022 +0530

    Add annotations to base.py

commit 906ab2ac21aa602c3e48e3fbbecf09487c535610
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Fri Apr 15 14:32:05 2022 +0530

    Added annotations to the main function in __init__.py

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

vlorentz added inline comments.
swh/indexer/metadata_dictionary/base.py
63

You can make it more specific than Any here

155

Optional and Union[..., None] is redundant, just use Optional[Dict]

swh/indexer/metadata_dictionary/cff.py
33

Why Optional and List?

swh/indexer/metadata_dictionary/codemeta.py
28

Technically, the dicts can be arbitrarily nested; so Optional[Dict[str, Any]] is more correct

swh/indexer/metadata_dictionary/maven.py
28

ditto

This revision now requires changes to proceed.Apr 19 2022, 8:17 AM
  • Revise changes to base.py WRT comments
  • Revise annotatiions to cff.py

Build is green

Patch application report for D7587 (id=27508)

Rebasing onto 4fe8cacf21...

Current branch diff-target is up to date.
Changes applied before test
commit 5f374579d2686ef7b62e74d140eaf6e995063732
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 14:16:02 2022 +0530

    Revise annotatiions to cff.py

commit fe87b377b24e0b3207588daf7d7c1b82736e9985
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 13:59:28 2022 +0530

    Revise changes to base.py WRT comments

commit f885ab3b45be36ac83511153eb0673c1d4fda727
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 03:44:22 2022 +0530

    Add annotations to maven.py

commit a7198bea6acfaeda8ff18be6ffe1e869966924e4
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 02:57:43 2022 +0530

    Add annotations to codemeta.py

commit dd6ca459b588b1b891d41cac57cdd8d0d2cab78f
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 02:37:00 2022 +0530

    Add some more annotations to base.py

commit 17072eb9a583fc609fa922bb8bd69532ea815d63
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 00:08:38 2022 +0530

    Add some more annotations to cff.py

commit f0b3401c1cee303fdabc9cc498c1b414d83f6e5e
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 23:53:27 2022 +0530

    Add some more annotations to base.py

commit 05f68f33fe7894d2791b9cb8260f5a33706f53a1
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 21:49:18 2022 +0530

    Add annotations to cff.py

commit 334d50fa304f05e7286fe07ddc174825d2000c43
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 20:15:59 2022 +0530

    Add annotations to cff.py

commit 74ad0bc730c908132c2a1f514e79ecd5b475af8b
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:23:38 2022 +0530

    Add annotations to base.py

commit e535236000516516ba64de1078c5e4c7ca72d7d8
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:06:56 2022 +0530

    Add annotations to base.py

commit 07bfc71539d4cb7fa3a3910eb2bdaa2c34b193cb
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 14:55:02 2022 +0530

    Add annotations to base.py

commit 906ab2ac21aa602c3e48e3fbbecf09487c535610
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Fri Apr 15 14:32:05 2022 +0530

    Added annotations to the main function in __init__.py

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

VickyMerzOwn added inline comments.
swh/indexer/metadata_dictionary/codemeta.py
28

I thought of that but then realised this would be more specific. ( I tried to avoid Any wherever I could)

swh/indexer/metadata_dictionary/maven.py
28

I could change it though. I haven't done type annotations before I came across the issue, so still learning things.

swh/indexer/metadata_dictionary/base.py
35

Use the same annotation here as you did below (this is the base class)

swh/indexer/metadata_dictionary/codemeta.py
28

that's true in general, but not here.

Revise annotations of and Format codemeta.py and maven.py

Build is green

Patch application report for D7587 (id=27526)

Rebasing onto 4fe8cacf21...

Current branch diff-target is up to date.
Changes applied before test
commit f3da534168357503e89a428f2eb75a99277b1220
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 19:30:42 2022 +0530

    Format codemeta.py and maven.py

commit 5f374579d2686ef7b62e74d140eaf6e995063732
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 14:16:02 2022 +0530

    Revise annotatiions to cff.py

commit fe87b377b24e0b3207588daf7d7c1b82736e9985
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 13:59:28 2022 +0530

    Revise changes to base.py WRT comments

commit f885ab3b45be36ac83511153eb0673c1d4fda727
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 03:44:22 2022 +0530

    Add annotations to maven.py

commit a7198bea6acfaeda8ff18be6ffe1e869966924e4
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 02:57:43 2022 +0530

    Add annotations to codemeta.py

commit dd6ca459b588b1b891d41cac57cdd8d0d2cab78f
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 02:37:00 2022 +0530

    Add some more annotations to base.py

commit 17072eb9a583fc609fa922bb8bd69532ea815d63
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 00:08:38 2022 +0530

    Add some more annotations to cff.py

commit f0b3401c1cee303fdabc9cc498c1b414d83f6e5e
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 23:53:27 2022 +0530

    Add some more annotations to base.py

commit 05f68f33fe7894d2791b9cb8260f5a33706f53a1
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 21:49:18 2022 +0530

    Add annotations to cff.py

commit 334d50fa304f05e7286fe07ddc174825d2000c43
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 20:15:59 2022 +0530

    Add annotations to cff.py

commit 74ad0bc730c908132c2a1f514e79ecd5b475af8b
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:23:38 2022 +0530

    Add annotations to base.py

commit e535236000516516ba64de1078c5e4c7ca72d7d8
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:06:56 2022 +0530

    Add annotations to base.py

commit 07bfc71539d4cb7fa3a3910eb2bdaa2c34b193cb
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 14:55:02 2022 +0530

    Add annotations to base.py

commit 906ab2ac21aa602c3e48e3fbbecf09487c535610
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Fri Apr 15 14:32:05 2022 +0530

    Added annotations to the main function in __init__.py

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

swh/indexer/metadata_dictionary/base.py
35

you forgot this

  • Revise annotations in base.py

Build is green

Patch application report for D7587 (id=27527)

Rebasing onto 4fe8cacf21...

Current branch diff-target is up to date.
Changes applied before test
commit 4435b65267b3a116cb5600938e242979da819a5c
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 21:37:30 2022 +0530

    Revise annotations in base.py

commit f3da534168357503e89a428f2eb75a99277b1220
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 19:30:42 2022 +0530

    Format codemeta.py and maven.py

commit 5f374579d2686ef7b62e74d140eaf6e995063732
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 14:16:02 2022 +0530

    Revise annotatiions to cff.py

commit fe87b377b24e0b3207588daf7d7c1b82736e9985
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 13:59:28 2022 +0530

    Revise changes to base.py WRT comments

commit f885ab3b45be36ac83511153eb0673c1d4fda727
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 03:44:22 2022 +0530

    Add annotations to maven.py

commit a7198bea6acfaeda8ff18be6ffe1e869966924e4
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 02:57:43 2022 +0530

    Add annotations to codemeta.py

commit dd6ca459b588b1b891d41cac57cdd8d0d2cab78f
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 02:37:00 2022 +0530

    Add some more annotations to base.py

commit 17072eb9a583fc609fa922bb8bd69532ea815d63
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Tue Apr 19 00:08:38 2022 +0530

    Add some more annotations to cff.py

commit f0b3401c1cee303fdabc9cc498c1b414d83f6e5e
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 23:53:27 2022 +0530

    Add some more annotations to base.py

commit 05f68f33fe7894d2791b9cb8260f5a33706f53a1
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 21:49:18 2022 +0530

    Add annotations to cff.py

commit 334d50fa304f05e7286fe07ddc174825d2000c43
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 20:15:59 2022 +0530

    Add annotations to cff.py

commit 74ad0bc730c908132c2a1f514e79ecd5b475af8b
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:23:38 2022 +0530

    Add annotations to base.py

commit e535236000516516ba64de1078c5e4c7ca72d7d8
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 15:06:56 2022 +0530

    Add annotations to base.py

commit 07bfc71539d4cb7fa3a3910eb2bdaa2c34b193cb
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Mon Apr 18 14:55:02 2022 +0530

    Add annotations to base.py

commit 906ab2ac21aa602c3e48e3fbbecf09487c535610
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Fri Apr 15 14:32:05 2022 +0530

    Added annotations to the main function in __init__.py

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

This revision is now accepted and ready to land.Apr 19 2022, 6:26 PM

Actually, one last thing: you need to squash your commits before landing this.

This revision now requires changes to proceed.Apr 19 2022, 8:34 PM

Build is green

Patch application report for D7587 (id=27529)

Rebasing onto 4fe8cacf21...

Current branch diff-target is up to date.
Changes applied before test
commit 270a2b3c74f4f66b9368bb21df84acf9c2287eba
Author: Satvik Vemuganti <vemugantisesha@iitbhilai.ac.in>
Date:   Fri Apr 15 14:32:05 2022 +0530

    Add annotations to the__init__.py, base.py, cff.py, codemeta.py and maven.py in metadata_dictionary

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

This revision is now accepted and ready to land.Apr 20 2022, 10:06 AM