origin model no longer allows to have type.
Related to f533f62bbf114cfcc29f7c72307c4dfbe99cf048
This will fix the current broken master build [1]
Similar to D2883
Related to D2815
[1] https://jenkins.softwareheritage.org/job/DJNL/job/tests/785/
Details
Details
- Reviewers
olasd - Group Reviewers
Reviewers - Commits
- rDJNL8ce4c0ba673e: tests: Adapt model according to latest change
tox
Diff Detail
Diff Detail
- Repository
- rDJNL Journal infrastructure
- Branch
- master
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 11390 Build 17246: tox-on-jenkins Jenkins Build 17245: arc lint + arc unit
Event Timeline
Comment Actions
Build is green
See https://jenkins.softwareheritage.org/job/DJNL/job/tox/415/ for more details.
Comment Actions
Oops, sorry, I thought of something.
I think we need an origin fixer to drop the potential types in the origin topic instead. @vlorentz, what do you think?
Comment Actions
Oops, sorry, I thought of something.
I think we need an origin fixer to drop the potential types in the origin topic instead. @vlorentz, what do you think?
I thought it was already there...
so something is off somewhere indeed.
Comment Actions
It's on origin_visit [1], not origin.
So yeah, probably as you mentioned, we need one.
[1] https://forge.softwareheritage.org/source/swh-journal/browse/master/swh/journal/fixer.py$236-239
Comment Actions
oops, i pushed that diff by mistake.
i think arcanist did more than what i asked... (well, what i had anticipated more like).
Still, i'll fix correctly the origin with a fixer in another diff.