Skip to content

Conversation

@britneywwc
Copy link
Contributor

@britneywwc britneywwc commented Dec 5, 2025

Note: uncomment PR action trigger before merging

Done

  • Added TIOBE config file: tiobe_config.txt
  • Added GH workflow to update TICS coverage: tics_coverage.yaml
    • The workflow runs coverage tests for python and js files on build-coverage. The coverage report is then uploaded as an artifact
    • The TICS action downloads the GH artifact and publishes it on TICS Viewer

QA

  • See TICS Analysis and coverage action passes, specifically TICS (22.x)
  • Check on TICS Viewer that the "Measured on" date is updated
  • Check that TQI score has increased from the screenshot before workflow integration

Issue / Card

Fixes WD-30653 and WD-30654

Screenshots

Screenshot 2025-12-12 at 3 18 38 PM

@webteam-app
Copy link

Comment on lines +2 to +6
on: pull_request
# on:
# schedule:
# - cron: "0 22 * * 5"
# workflow_dispatch:
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For QA. Remove pull_request and uncomment before merging

@codecov
Copy link

codecov bot commented Dec 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.20%. Comparing base (068b523) to head (889f4d5).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2117   +/-   ##
=======================================
  Coverage   77.20%   77.20%           
=======================================
  Files          16       16           
  Lines        1676     1676           
=======================================
  Hits         1294     1294           
  Misses        382      382           
Flag Coverage Δ
python 77.20% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@britneywwc britneywwc force-pushed the tiobe-config branch 27 times, most recently from 509e8fb to 6fe1d6a Compare December 9, 2025 09:38
Copy link
Member

@immortalcodes immortalcodes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this PR. The dashboard looks good, I have mostly exploratory comments.

Comment on lines +40 to +42
coverage run --data-file=.coverage_db --source=webapp -m unittest discover tests &&
coverage report --data-file=.coverage_db &&
coverage xml --data-file=.coverage_db -o coverage/coverage.xml
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you help me understand this commands better, as far as I know the coverage is already calculated in other test workflows, what's different here?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These commands run coverage on python code locally, and generate coverage outpul in xml format.
The coverage calculated in other workflows is on the PR itself. The reason for doing it here is on the main branch of repo, which is going to run every week.

@@ -0,0 +1,12 @@
'DIR' =>
"/webapp/" ||
"/templates/" ||
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see templates on the dashboard, is that expected?

Image

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not expected, it should be included.
I have updated the tiobe config, so it should now be generated.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants