Skip to content

Python local flow use fasttc - #22245

Closed
chanel-y wants to merge 1600 commits into
github:mainfrom
microsoft:python-localFlow-use-fasttc
Closed

Python local flow use fasttc#22245
chanel-y wants to merge 1600 commits into
github:mainfrom
microsoft:python-localFlow-use-fasttc

Conversation

@chanel-y

Copy link
Copy Markdown
Contributor

Resolving performance blowup from DCA run

bdrodes and others added 30 commits January 15, 2026 15:23
…iewing the unit tests and conversations about how to handle some of the fp/fn cases observed. Updated the unit tests to use InlineExpectationsTestQuery.ql so it is easier to detect FP/FNs.
…eds to be generally reassessed but recent test changes alter the expected results.
…efinitions had gaps resulting in false positives and inconsistencies (inconsistent as to what is a guard and what is a function that does a leap year check).
…anel

PS: API graph coverage improvements
…n on LeapYear.qll. We may need to end up moving new capabilities for this query into LeapYear.qll for other queries in the future. For now, focusing just on improvements to the one query. The only remaining dependency is the TimeConversionFunction class, which is currently used by another query.
…ions are now assumed to be a check of the result, i.e., a valid leap year check.
PS: Fix example in SQL injection documentation
Compatible with the latest released version of the CodeQL CLI
Copilot AI review requested due to automatic review settings July 28, 2026 19:10
@chanel-y
chanel-y requested review from a team as code owners July 28, 2026 19:10

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

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.

9 participants