Skip to content

Improve Workflow CI#22

Open
shiningflash wants to merge 1 commit intomainfrom
workflows-improvement
Open

Improve Workflow CI#22
shiningflash wants to merge 1 commit intomainfrom
workflows-improvement

Conversation

@shiningflash
Copy link
Owner

This pull request updates the continuous integration workflow in .github/workflows/ci.yml to improve caching, streamline dependency management, and make Allure test reporting more robust and efficient. The changes focus on optimizing CI performance, enhancing the reliability of test report publishing, and improving the formatting of test report comments.

CI performance and dependency management:

  • Added caching for Allure CLI npm dependencies and Python dependencies using actions/cache@v3, which speeds up workflow runs by avoiding redundant installs.
  • Moved the installation of Allure CLI to a separate step after caching, ensuring the cache is used effectively.

Allure report generation and publishing:

  • Improved fetching and restoring of Allure report history by checking out the entire allure-report branch and restoring the previous working branch, which helps maintain report continuity between runs.
  • Added a conditional check to only deploy the Allure report to GitHub Pages if the report was actually generated, preventing unnecessary deployments.

Test report comment formatting:

  • Refactored the construction of the test report body in the GitHub PR comment to use an array joined by newlines, improving readability and maintainability of the message formatting.

Permissions update:

  • Granted the workflow checks: write permission, which may be needed for publishing or updating check runs associated with the CI process.

@github-actions
Copy link

github-actions bot commented Nov 9, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
src/core
   __init__.py00100% 
   config.py2322 91%
   logger.py220100% 
   requests.py1544 73%
   retry.py2111 95%
   security.py2622 92%
   validators.py400100% 
src/core/exceptions
   __init__.py40100% 
   api.py2911 97%
   decorators.py2377 70%
   resource.py130100% 
src/schemas
   __init__.py00100% 
   contacts.py140100% 
   errors.py170100% 
   messages.py3988 79%
   webhook.py80100% 
src/sdk
   __init__.py00100% 
   client.py4222 95%
src/sdk/features
   __init__.py00100% 
   contacts.py4733 94%
   messages.py441010 77%
src/server
   __init__.py00100% 
   app.py2633 88%
TOTAL4534391% 

Tests Skipped Failures Errors Time
55 0 💤 0 ❌ 0 🔥 7.223s ⏱️

@github-actions
Copy link

github-actions bot commented Nov 9, 2025

📊 Test Report ✅ All tests passed

Coverage trends • Module breakdown • Failure analysis • Execution timeline

View Allure Dashboard

⚠️ NB: If the dashboard doesn't update immediately, try a hard reload (Cmd + Shift + R on Mac) or open in incognito mode.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant