feat: add pytest as a dev dependency and a github CI workflow
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 62/100
Research direction
Start in the project's pyproject.toml and add pytest under the Poetry dev dependency group, then look at the existing .github/workflows directory for a Python workflow to copy. Add a workflow that installs dependencies and runs pytest, triggered on pull requests that touch Python files via a paths filter. The test file added in PR #257 is the one to run. Done when the workflow passes on a PR that changes a .py file.
Written by the indexing model from the issue text.
Description
nice of you to add a test, but currently there is no runner of this in CI or otherwise.
this would be good to have, should we have another PR adding this cleanly?
that could include this test file, dev dependency added to pyproject for poetry and a github workflow that would run on every PR touching the python files.
Originally posted by @kyteinsky in https://github.com/nextcloud/context_agent/pull/257#discussion_r4270092295
- Dominant language
- Python
- Stars
- 26
- Forks
- 20
- Avg merge
- 2d 9h
- Merged PRs (30d)
- 4
Getting set up
- Ships a Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from nextcloud/context_agent
-
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
nextcloud/context_agent#242 · 1 comment ·
Maintainers usually reply within 1 day
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
nextcloud/context_agent#240 · 1 comment ·
Maintainers usually reply within 1 day
-
find_person_in_contacts fails with Niquests content argument and cannot serialize vCard AddressOpenbug
Difficulty 2/5 Half a day Newbie friendliness 76/100
nextcloud/context_agent#217 · 3 comments ·
Maintainers usually reply within 1 day
-
bug
Difficulty 4/5 3-5 days Newbie friendliness 22/100
nextcloud/context_agent#271 ·
Maintainers usually reply within 1 day
-
bug priority normal
Difficulty 4/5 3-5 days Newbie friendliness 22/100
nextcloud/context_agent#269 ·
Maintainers usually reply within 1 day
All issues in nextcloud/context_agent
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
common-workflow-language/cwl-utils#477 ·
Maintainers usually reply within 1 day
-
Add dependabotOpenfeat
Difficulty 2/5 Under an hour Newbie friendliness 72/100
open-nudge/opentemplate#204 ·
Maintainers usually reply within 1 day
-
github-stale-ci-pr-closer: KV state is double-wrapped, never persists, and hits the depth-32 limitPossibly taken @VascoSch92 claimed this today. Openpriority:medium ready-for-dev
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
OpenHands/extensions#781 ·
Maintainers usually reply within 1 day
-
bug good first issue
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 1 day
-
curriculum documentation quality
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
githubnext/gh-aw-workshop#4638 ·
Maintainers usually reply within 1 day