[PR #315] [MERGED] #22 Setup coverage for CI/CD, some pragma no cover comments #379

Closed
opened 2026-02-27 19:17:58 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/EddieTheCubeHead/Stagnum/pull/315
Author: @EddieTheCubeHead
Created: 5/18/2024
Status: Merged
Merged: 5/20/2024
Merged by: @EddieTheCubeHead

Base: masterHead: 22-setup-coverage-tests-for-backend


📝 Commits (10+)

  • 26515ce #22 Setup coverage for CI/CD, some pragma no cover comments
  • abd694e #22 fix test folder name in cicd workflow
  • f70eb92 #22 allow a larger timeframe for test
  • 06c738e #22 Experiment with different coverage workflow params
  • 7132e3e #22 More fix attempts
  • 9a53d3f #22 more fix attempts
  • dc67464 #22 Grant write permissions to pull requests for backend cicd action
  • b5f2e70 #22 maybe we need to change action ordering?
  • 6552fbd #22 try putting this here
  • c6889e5 #22 omit main.py and logging_config.py from coverage report

📊 Changes

9 files changed (+35 additions, -25 deletions)

View changed files

📝 .github/workflows/backend-ci.yml (+13 -6)
📝 server/pyproject.toml (+14 -1)
📝 server/src/api/application.py (+1 -1)
📝 server/src/api/auth/tasks.py (+1 -2)
📝 server/src/api/common/dependencies.py (+2 -5)
📝 server/src/database/database_connection.py (+1 -1)
📝 server/src/main.py (+0 -7)
📝 server/test/framework_features.py (+1 -1)
📝 server/test_requirements.txt (+2 -1)

📄 Description

No description provided


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/EddieTheCubeHead/Stagnum/pull/315 **Author:** [@EddieTheCubeHead](https://github.com/EddieTheCubeHead) **Created:** 5/18/2024 **Status:** ✅ Merged **Merged:** 5/20/2024 **Merged by:** [@EddieTheCubeHead](https://github.com/EddieTheCubeHead) **Base:** `master` ← **Head:** `22-setup-coverage-tests-for-backend` --- ### 📝 Commits (10+) - [`26515ce`](https://github.com/EddieTheCubeHead/Stagnum/commit/26515ce4e6690a51d953eb8e178f12cb5781af97) #22 Setup coverage for CI/CD, some pragma no cover comments - [`abd694e`](https://github.com/EddieTheCubeHead/Stagnum/commit/abd694e006936ab08ae5e81b0b8a01ae3b4f4490) #22 fix test folder name in cicd workflow - [`f70eb92`](https://github.com/EddieTheCubeHead/Stagnum/commit/f70eb920b84310e0f04c2d7c97e5309b28eb3f58) #22 allow a larger timeframe for test - [`06c738e`](https://github.com/EddieTheCubeHead/Stagnum/commit/06c738eeb79aba9310103f4132e54f5476429973) #22 Experiment with different coverage workflow params - [`7132e3e`](https://github.com/EddieTheCubeHead/Stagnum/commit/7132e3e9def36249c41f32e4961f822b150a4312) #22 More fix attempts - [`9a53d3f`](https://github.com/EddieTheCubeHead/Stagnum/commit/9a53d3f0c28d7d0425673d1c7b5cedca71f238e9) #22 more fix attempts - [`dc67464`](https://github.com/EddieTheCubeHead/Stagnum/commit/dc67464d12bf692807b6a66c4ec58cfbca664201) #22 Grant write permissions to pull requests for backend cicd action - [`b5f2e70`](https://github.com/EddieTheCubeHead/Stagnum/commit/b5f2e70abdac5275b8ccce709b7ec93b591420c1) #22 maybe we need to change action ordering? - [`6552fbd`](https://github.com/EddieTheCubeHead/Stagnum/commit/6552fbd99fa05398543bd28ae540348994858bc4) #22 try putting this here - [`c6889e5`](https://github.com/EddieTheCubeHead/Stagnum/commit/c6889e5f80fed184506aedff0105222f61594831) #22 omit main.py and logging_config.py from coverage report ### 📊 Changes **9 files changed** (+35 additions, -25 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/backend-ci.yml` (+13 -6) 📝 `server/pyproject.toml` (+14 -1) 📝 `server/src/api/application.py` (+1 -1) 📝 `server/src/api/auth/tasks.py` (+1 -2) 📝 `server/src/api/common/dependencies.py` (+2 -5) 📝 `server/src/database/database_connection.py` (+1 -1) 📝 `server/src/main.py` (+0 -7) 📝 `server/test/framework_features.py` (+1 -1) 📝 `server/test_requirements.txt` (+2 -1) </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-27 19:17:58 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/Stagnum#379
No description provided.