Skip to content

TICS: Improve compiler warning scores#2248

Open
britneywwc wants to merge 4 commits intomainfrom
tics-compiler-score
Open

TICS: Improve compiler warning scores#2248
britneywwc wants to merge 4 commits intomainfrom
tics-compiler-score

Conversation

@britneywwc
Copy link
Contributor

@britneywwc britneywwc commented Feb 5, 2026

Done

  • Address compiler warnings for JS files under /static/js/
  • Majority of the change is adding type associations for variables. Minor changes made to the functionalities.
  • Initial Compiler Warning score: 13.53%
  • Initial static file Compiler Warning score: 10.09%
  • Final Compiler Warning score: 43.38%
  • Final static file Compiler Warning score: 34.83%

QA

  • Go to demos
  • Check that JS functionalities are working
  • Go to nav, click on careers and see that roles are showing as expected
  • Go to TiCS dashboard, compare the updated Compiler Warning score with the screenshot below

Issue / Card

Fixes WD-32027

Screenshots

Before changes
Screenshot 2026-02-05 at 4 01 52 PM

@webteam-app
Copy link

@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.08%. Comparing base (25ca88a) to head (f3c4765).
⚠️ Report is 10 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2248      +/-   ##
==========================================
+ Coverage   78.88%   79.08%   +0.20%     
==========================================
  Files          16       16              
  Lines        1804     1822      +18     
==========================================
+ Hits         1423     1441      +18     
  Misses        381      381              
Flag Coverage Δ
python 79.08% <ø> (+0.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 marked this pull request as draft February 5, 2026 08:16
@britneywwc britneywwc force-pushed the tics-compiler-score branch 5 times, most recently from 6681638 to 39ecf2c Compare February 6, 2026 08:45
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.

3 participants