Since the function extract_regex_objs was moved to swh.model.from_disk the scanner should use the one in the model in order to remove the duplicate code between the two.
Tests changed according to the new function.
Fixes T2679
Details
Details
- Reviewers
vlorentz - Group Reviewers
Reviewers - Maniphest Tasks
- T2679: Use the `swh.model` version of `extract_regex_objs`
- Commits
- rDTSCNb3256c87728e: use 'extract_regex_objs' from swh.model
Diff Detail
Diff Detail
- Repository
- rDTSCN Code scanner
- Branch
- reflect-model-changes
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 20290 Build 31495: Phabricator diff pipeline on jenkins Jenkins console · Jenkins Build 31494: arc lint + arc unit
Event Timeline
Comment Actions
Build is green
Patch application report for D5359 (id=19197)
Rebasing onto eb04424434...
Current branch diff-target is up to date.
Changes applied before test
commit 7b06a22ac5c910b496d5d4c7106f08220ba2dd2e
Author: Daniele Serafini <me@danieleserafini.eu>
Date: Thu Mar 25 17:20:05 2021 +0100
use 'extract_regex_objs' from swh.modelSee https://jenkins.softwareheritage.org/job/DTSCN/job/tests-on-diff/113/ for more details.
| swh/scanner/scanner.py | ||
|---|---|---|
| 218 | why not root_path.encode()? | |
| swh/scanner/scanner.py | ||
|---|---|---|
| 218 | honestly i didn't think about it, but probably it's better to convert all the string with .encode() as you said | |
| swh/scanner/scanner.py | ||
|---|---|---|
| 218 | oh, i just noticed that i don't need to convert the root_path there, thanks! | |
Comment Actions
Build is green
Patch application report for D5359 (id=19209)
Rebasing onto eb04424434...
Current branch diff-target is up to date.
Changes applied before test
commit b3256c87728eef4ed44c91c3c8daa59960debbbd
Author: Daniele Serafini <me@danieleserafini.eu>
Date: Thu Mar 25 17:20:05 2021 +0100
use 'extract_regex_objs' from swh.modelSee https://jenkins.softwareheritage.org/job/DTSCN/job/tests-on-diff/114/ for more details.