Skip to content

Conversation

@aschackmull
Copy link
Contributor

The name ControlFlow wasn't quite clear enough. The new name is a bit long, but it's a lot clearer.

@aschackmull aschackmull requested a review from a team as a code owner October 23, 2025 07:11
Copilot AI review requested due to automatic review settings October 23, 2025 07:11
@aschackmull aschackmull requested review from a team as code owners October 23, 2025 07:11
@aschackmull aschackmull added the no-change-note-required This PR does not need a change note label Oct 23, 2025
Copy link
Contributor

Copilot AI 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 renames the ControlFlow library to ControlFlowReachability across Java and C# codebases to improve clarity of the library's purpose.

Key Changes:

  • Renamed import alias from Cf to direct imports without aliasing
  • Updated module references from ControlFlow to ControlFlowReachability

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

File Description
java/ql/lib/semmle/code/java/dataflow/Nullness.qll Removed import alias and updated module references to use new library name
java/ql/lib/semmle/code/java/controlflow/ControlFlowReachability.qll Updated import and module instantiation to use new library name
csharp/ql/lib/semmle/code/csharp/dataflow/Nullness.qll Removed import alias and updated module references to use new library name
csharp/ql/lib/semmle/code/csharp/controlflow/ControlFlowReachability.qll Updated import to use new library name

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@aschackmull aschackmull merged commit f6cddc9 into github:main Oct 23, 2025
48 of 50 checks passed
@aschackmull aschackmull deleted the shared/rename-cfg-reach branch October 23, 2025 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C# Java no-change-note-required This PR does not need a change note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants