Skip to content

chore: exclude build output from analysis - #24

Closed
smiggleworth wants to merge 1 commit into
mainfrom
chore/exclude-askr-build-analysis
Closed

chore: exclude build output from analysis#24
smiggleworth wants to merge 1 commit into
mainfrom
chore/exclude-askr-build-analysis

Conversation

@smiggleworth

Copy link
Copy Markdown
Contributor

Summary

  • exclude generated .askr client bundles from static analysis
  • keep analysis scoped to the 43 authored source files

Validation

  • askr analyze --check
  • npm run check

Copilot AI review requested due to automatic review settings July 28, 2026 17:12
@smiggleworth

Copy link
Copy Markdown
Contributor Author

Closing because the global CLI was updated to 0.0.15 and the repository cleanup will remain local; no package-level exclusion is needed.

@smiggleworth
smiggleworth deleted the chore/exclude-askr-build-analysis branch July 28, 2026 17:13

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the repository’s Askr CLI analysis configuration to ignore generated build artifacts so static analysis focuses on authored source files.

Changes:

  • Added an askr.analyze.exclude configuration block in package.json.
  • Configured analysis to exclude files under .askr/**.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread package.json
Comment on lines +60 to +64
"askr": {
"analyze": {
"exclude": [
".askr/**"
]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants