Repository navigation
diff: add -p and -F to show function lines in hunk headers - #323
Conversation
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing. Review the Copilot code review billing settings.
bd5dce2 to
76fc0f9
Compare
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing. Review the Copilot code review billing settings.
76fc0f9 to
f8d86e9
Compare
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing. Review the Copilot code review billing settings.
f8d86e9 to
ba47aa3
Compare
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing. Review the Copilot code review billing settings.
Should make test function-line-vs-leading-space pass.
ba47aa3 to
8e582da
Compare
There was a problem hiding this comment.
Warning
Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.
Copilot review overview
3 open issues
Review effort: Lite
| let mut function_lines = FunctionLineFinder::new(expected, ¶ms.function_line_patterns); | ||
| for result in diff_results { | ||
| let function_line = function_lines | ||
| .as_mut() | ||
| .and_then(|finder| finder.find(result.line_number_expected)); |
| let mut function_lines = FunctionLineFinder::new(expected, ¶ms.function_line_patterns); | ||
| for result in diff_results { | ||
| let function_line = function_lines | ||
| .as_mut() | ||
| .and_then(|finder| finder.find(result.line_number_expected as usize)); |
| pub fn new(old_text: &'a [u8], patterns: &'a [Bre]) -> Option<Self> { | ||
| if patterns.is_empty() { | ||
| return None; | ||
| } | ||
| Some(Self { | ||
| lines: old_text.split(|&byte| byte == b'\n').collect(), | ||
| patterns, | ||
| next_line: 0, | ||
| last_match: None, | ||
| }) | ||
| } |
|
GNU diffutils testsuite comparison: |
Stack created with GitHub Stacks CLI • Give Feedback 💬