oblachno-oss/devx
Public Access
3
0
Fork 0
Code Issues 52 Pull Requests Actions Packages Projects Releases 9 Wiki Activity

DEVX-119: feat: add GiteaClient repo variable methods and speed up pytest-cov #182

Merged
emil merged 0 commits from DEVX-119-add-repo-variable-methods-and-speed-up-tests into master 2026-07-07 11:57:08 +00:00
Conversation 6 Commits - Files Changed -
emil commented 2026-07-07 11:29:19 +00:00
Owner
Copy Link
Copy Source

Summary

  • GiteaClient.get_repo_variable() — read Gitea Actions repo variables, returns None on 404
  • GiteaClient.set_repo_variable() — idempotent create-or-update (PATCH then POST on 404)
  • 7 new tests with 100% coverage (get/set, 404 handling, non-404 reraise, create-on-404)
  • pytest-cov speedup: added -n auto (xdist) and removed -v — reduces from ~13s to ~6s

Test plan

  • 1735 tests pass, 100% coverage
  • lint-all passes
  • check-test-speed: 3.18s (under 6s limit)
  • pytest-cov: 6.27s (was 12.93s)
  • CI pipeline passes
## Summary - **GiteaClient.get_repo_variable()** — read Gitea Actions repo variables, returns None on 404 - **GiteaClient.set_repo_variable()** — idempotent create-or-update (PATCH then POST on 404) - **7 new tests** with 100% coverage (get/set, 404 handling, non-404 reraise, create-on-404) - **pytest-cov speedup**: added `-n auto` (xdist) and removed `-v` — reduces from ~13s to ~6s ## Test plan - [x] 1735 tests pass, 100% coverage - [x] lint-all passes - [x] check-test-speed: 3.18s (under 6s limit) - [x] pytest-cov: 6.27s (was 12.93s) - [ ] CI pipeline passes
emil reviewed 2026-07-07 11:29:36 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: OK
  • Commit conventions: OK

No issues found by automated checks.

Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil added the ready-to-merge label 2026-07-07 11:42:05 +00:00
emil reviewed 2026-07-07 11:42:47 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: OK
  • Commit conventions: OK

No issues found by automated checks.

Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil force-pushed DEVX-119-add-repo-variable-methods-and-speed-up-tests from c3428efb4e to ec460b1bf9 2026-07-07 11:44:57 +00:00 Compare
emil reviewed 2026-07-07 11:45:13 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: OK
  • Tests: OK
  • Commit conventions: OK

No issues found by automated checks.

Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: OK - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil force-pushed DEVX-119-add-repo-variable-methods-and-speed-up-tests from ec460b1bf9 to 5ede6e8e0b 2026-07-07 11:48:19 +00:00 Compare
emil reviewed 2026-07-07 11:48:34 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: OK
  • Tests: OK
  • Commit conventions: OK

No issues found by automated checks.

Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: OK - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil force-pushed DEVX-119-add-repo-variable-methods-and-speed-up-tests from 5ede6e8e0b to c365547893 2026-07-07 11:55:47 +00:00 Compare
emil reviewed 2026-07-07 11:56:03 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: OK
  • Tests: OK
  • Commit conventions: OK

No issues found by automated checks.

Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: OK - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
kireto approved these changes 2026-07-07 11:57:00 +00:00
kireto left a comment
Member
Copy Link
Copy Source

Auto-approved: all CI checks passed (quality, pr-review, release-dry-run).

Auto-approved: all CI checks passed (quality, pr-review, release-dry-run).
emil merged commit f98534ebe2 into master 2026-07-07 11:57:08 +00:00
emil deleted branch DEVX-119-add-repo-variable-methods-and-speed-up-tests 2026-07-07 11:57:10 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
emil
kireto
Labels
Clear labels
bug
Something is not working
 ci-improvement
CI workflow improvements
 doc-improvement
Documentation improvements
 feedback
Issues from subagent or agent feedback
 ready-to-merge
tooling
Tool-related feedback or improvements
 workflow-improvement
Workflow alignment or pattern improvements
No labels ready-to-merge
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
emil (emil User) emo kireto
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/devx#182