Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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

Merged
cklin merged 4 commits intomainfromcklin/diff-informed-feature-enable
Jul 17, 2025

Conversation

cklin
Copy link
Contributor

@cklincklin commentedJul 14, 2025
edited
Loading

This PR updatesFeature.DiffInformedQueries to be enabled on GHES versions 3.19 or higher.

Merge / deployment checklist

  • Confirm this change is backwards compatible with existing workflows.
  • Confirm thereadme has been updated if necessary.
  • Confirm thechangelog has been updated if necessary.

@cklincklin marked this pull request as ready for reviewJuly 14, 2025 19:10
@CopilotCopilotAI review requested due to automatic review settingsJuly 14, 2025 19:10
@cklincklin requested a review froma team as acode ownerJuly 14, 2025 19:10
Copilot

This comment was marked as outdated.

@cklincklin requested a review frommbgJuly 14, 2025 19:11
@cklincklinforce-pushed thecklin/diff-informed-feature-enable branch from91304fa to56692c1CompareJuly 16, 2025 18:10
@cklincklin requested a review fromCopilotJuly 16, 2025 18:12
Copy link
Contributor

@CopilotCopilotAI left a 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 forFeature.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
FileDescription
src/feature-flags.tsUpdates default value for DiffInformedQueries feature from false to true
src/diff-informed-analysis-utils.tsAdds GHES version checking logic to restrict feature to 3.19+
src/diff-informed-analysis-utils.test.tsAdds comprehensive test cases for diff-informed analysis functionality
lib/feature-flags.jsCompiled JavaScript version of the TypeScript feature flag changes
lib/diff-informed-analysis-utils.test.jsCompiled JavaScript version of the TypeScript test file
lib/diff-informed-analysis-utils.jsCompiled JavaScript version of the TypeScript utility changes

@cklincklinforce-pushed thecklin/diff-informed-feature-enable branch from56692c1 to4447fc3CompareJuly 16, 2025 18:18
mbg
mbg previously approved these changesJul 17, 2025
Copy link
Member

@mbgmbg left a 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.

@cklincklinforce-pushed thecklin/diff-informed-feature-enable branch from4447fc3 to6a49a8cCompareJuly 17, 2025 13:17
@cklincklin requested a review frommbgJuly 17, 2025 13:37
@cklincklin merged commit7710ed1 intomainJul 17, 2025
282 checks passed
@cklincklin deleted the cklin/diff-informed-feature-enable branchJuly 17, 2025 15:21
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

Copilot code reviewCopilotCopilot left review comments

@mbgmbgmbg approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@cklin@mbg

[8]ページ先頭

©2009-2025 Movatter.jp