jackfirth / resyntax

93%

master: 94%

LAST BUILD ON BRANCH copilot/refactor-test-imports

branch: copilot/refactor-test-imports

CHANGE BRANCH

x

Reset

• copilot/refactor-test-imports
• HEAD
• autofix-24-1
• autopilot-blocking
• autopilot-workflow
• ci-debug
• cli-json
• compile-test
• copilot/add-analyzer-load-tests
• copilot/add-analyzer-timeout-option
• copilot/add-expanded-id-table-utility
• copilot/add-function-expression-analyzer
• copilot/add-in-syntax-paths-utility
• copilot/add-printf-without-specifiers-rule
• copilot/add-resyntax-fix-json-output
• copilot/add-string-representation-syntax-paths-again
• copilot/add-syntax-path-filter-utility
• copilot/add-syntax-property-extraction-apis
• copilot/add-warning-rule-support
• copilot/allow-custom-analyzers-in-rules
• copilot/configurable-analyzer-timeout
• copilot/consolidate-legacy-migration-rules
• copilot/extract-let-from-arguments
• copilot/fix-8175575-328049080-6434af77-80e7-42db-a2ea-55d3c2dafeba
• copilot/fix-aggressive-when-merging
• copilot/fix-broken-code-replacements
• copilot/fix-error-to-raise-arguments-error
• copilot/fix-identifier-usage-analyzer
• copilot/fix-improper-syntax-paths
• copilot/fix-syntax-path-logging
• copilot/fix-unwanted-suggestion
• copilot/handle-multiple-variables-in-rule
• copilot/improve-test-coverage-file-group
• copilot/improve-test-coverage-github
• copilot/move-rules-to-let-replacement
• copilot/refactor-analyzers-reuse-table-code
• copilot/refactor-rules-suites-analyzers
• copilot/refactor-sequence-tail-replacement
• copilot/refactor-source-analyze-analyzers
• copilot/remove-matching-comparator-file
• copilot/remove-proper-syntax-path-predicate
• copilot/skip-failing-analyzers
• copilot/skip-push-workflows-for-copilot
• copilot/split-formatting-preservation-tests
• copilot/store-visited-paths
• copilot/suggest-using-define-syntax-parse-rule
• copilot/update-identifier-usage-analyzer
• copilot/use-module-lexer-for-comments
• coverage
• coverage-fixing
• coverage-loops
• coverage-workflow
• fix-comment-reading
• jackfirth-patch-1
• jackfirth-patch-1-1
• jackfirth-patch-2
• let-replacement
• manual-testing
• manual-workflow
• master
• pr-analysis
• run-rule

Build # #127

Build Type

Pull #703

cover

Committed by Copilot

Commit Message

Simplify check-suite-analysis by using reysntax-analyze-for-properties-only

Instead of reimplementing the analysis logic with error handling and
#lang racket checks, delegate to the existing reysntax-analyze-for-
properties-only function which already handles all these cases.

- Replaced custom analysis logic with call to reysntax-analyze-for-properties-only
- Removed unnecessary resyntax/private/analysis import
- Simplified implementation while maintaining same functionality

Co-authored-by: jackfirth <8175575+jackfirth@users.noreply.github.com>

Pull Request Pull Request #703: Support importing analyzers directly in test require: statements

Run Details

28 of 32 new or added lines in 5 files covered. (87.5%)

1 existing line in 1 file now uncovered.

14440 of 15545 relevant lines covered (92.89%)

0.93 hits per line