Skip to content

Fix any type usage in the saas/ folder#5934

Open
jbrunton96 wants to merge 1 commit intomainfrom
fix-any
Open

Fix any type usage in the saas/ folder#5934
jbrunton96 wants to merge 1 commit intomainfrom
fix-any

Conversation

@jbrunton96
Copy link
Contributor

Description of Changes

Ages ago I made #4835 to try and fix all the any type usage in the system but never got it finished, and there were just too many to review and ensure it still worked. There's even more now.

My new tactic is to fix folder by folder. This fixes the any typing in the saas/ folder, and also enables no-unnecessary-type-assertion, which really helps reduce pointless as casts that AI generates when the type is already known. I hope to expand both of these to the rest of the folders soon, but one folder is better than none.

@jbrunton96 jbrunton96 self-assigned this Mar 13, 2026
@stirlingbot stirlingbot bot added the Front End Issues or pull requests related to front-end development label Mar 13, 2026
@jbrunton96 jbrunton96 marked this pull request as ready for review March 13, 2026 17:02
@dosubot dosubot bot added the size:L This PR changes 100-499 lines ignoring generated files. label Mar 13, 2026
@stirlingbot
Copy link
Contributor

stirlingbot bot commented Mar 13, 2026

🚀 V2 Auto-Deployment Complete!

Your V2 PR with embedded architecture has been deployed!

🔗 Direct Test URL (non-SSL) http://23.22.230.180:5934

🔐 Secure HTTPS URL: https://5934.ssl.stirlingpdf.cloud

This deployment will be automatically cleaned up when the PR is closed.

🔄 Auto-deployed for approved V2 contributors.

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

Labels

Front End Issues or pull requests related to front-end development size:L This PR changes 100-499 lines ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant