- Notifications
You must be signed in to change notification settings - Fork373
Enable Feature.DiffInformedQueries#2970
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Uh oh!
There was an error while loading.Please reload this page.
91304fa
to56692c1
CompareThere was a problem hiding this comment.
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 enables theFeature.DiffInformedQueries
feature by default and restricts it to GHES versions 3.19 or higher. The primary goal is to make diff-informed queries available to more users while ensuring compatibility with supported GHES versions.
- Changed the default value for
Feature.DiffInformedQueries
fromfalse
totrue
- Added GHES version checking to restrict the feature to version 3.19 or higher
- Added comprehensive test coverage for the new GHES version validation logic
Reviewed Changes
Copilot reviewed 6 out of 9 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
src/feature-flags.ts | Updates default value for DiffInformedQueries feature from false to true |
src/diff-informed-analysis-utils.ts | Adds GHES version checking logic to restrict feature to 3.19+ |
src/diff-informed-analysis-utils.test.ts | Adds comprehensive test cases for diff-informed analysis functionality |
lib/feature-flags.js | Compiled JavaScript version of the TypeScript feature flag changes |
lib/diff-informed-analysis-utils.test.js | Compiled JavaScript version of the TypeScript test file |
lib/diff-informed-analysis-utils.js | Compiled JavaScript version of the TypeScript utility changes |
56692c1
to4447fc3
CompareThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
LGTM, thanks for changing this and adding tests! Just one minor question / suggestion.
Uh oh!
There was an error while loading.Please reload this page.
4447fc3
to6a49a8c
Compare7710ed1
intomainUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
This PR updates
Feature.DiffInformedQueries
to be enabled on GHES versions 3.19 or higher.Merge / deployment checklist