Exclude atom tests data from pre-commit processing and fix typo
ClosedPublic
Actions

Authored by anlambert on Oct 19 2022, 2:38 PM.

Details

Reviewers

Group Reviewers

Commits

rDDEP9f8535874de7: docs/api/metadata: Fix typo detected by codespell
rDDEP85015daece77: pre-commit: exclude atom tests data

Test Plan

Due to the recent bump of codespell, new spelling errors were reported.

pre-commit run --all-files
trim trailing whitespace.................................................Passed
check json...............................................................Passed
check yaml...............................................................Passed
flake8...................................................................Passed
Check source code spelling...............................................Failed
- hook id: codespell
- exit code: 65

swh/deposit/tests/data/atom/codemeta-sample.xml:36: Projet ==> Project
swh/deposit/tests/data/atom/codemeta-sample.xml:37: projet ==> project
swh/deposit/tests/data/atom/codemeta-sample.xml:42: Projet ==> Project
swh/deposit/tests/data/atom/codemeta-sample.xml:43: Projet ==> Project
docs/api/metadata.rst:107: Projet ==> Project

mypy.....................................................................Passed
isort....................................................................Passed
black....................................................................Passed
make: *** [../Makefile.python:28: check] Error 1

Diff Detail

Repository

rDDEP Push deposit

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 32468
Build 50850: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 50849: arc lint + arc unit

Event Timeline

anlambert created this revision.Oct 19 2022, 2:38 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptOct 19 2022, 2:38 PM

Build is green

Patch application report for D8746 (id=31523)

Rebasing onto 5669fc8298...

Current branch diff-target is up to date.

Changes applied before test

commit 9f8535874de7ebf237add7a0bd62acc081e57e4e
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Wed Oct 19 14:36:18 2022 +0200

    docs/api/metadata: Fix typo detected by codespell

commit 85015daece772a379f8560134a4bafd6037ecd3c
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Wed Oct 19 14:35:45 2022 +0200

    pre-commit: exclude atom tests data

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

Harbormaster completed remote builds in B32468: Diff 31523.Oct 19 2022, 2:44 PM

anlambert requested review of this revision.Oct 19 2022, 2:44 PM

ardumont accepted this revision.Oct 19 2022, 2:56 PM

This revision is now accepted and ready to land.Oct 19 2022, 2:56 PM

Closed by commit rDDEP85015daece77: pre-commit: exclude atom tests data (authored by anlambert). · Explain WhyOct 19 2022, 2:57 PM

This revision was automatically updated to reflect the committed changes.

anlambert added a commit: rDDEP85015daece77: pre-commit: exclude atom tests data.

anlambert added a commit: rDDEP9f8535874de7: docs/api/metadata: Fix typo detected by codespell.