On Sat, Feb 01, 2025 at 12:11:58PM +0530, Meet Soni wrote: Please drop the `refactor()` bit from the commit subject, we don't use these prefixes here. You also did the same in a later commit. > Rename the function `omit_name_by_refspec()` to > `refname_matches_negative_refspec_item()` to provide clearer intent. > The previous function name was vague and did not accurately describe its > purpose. By using `refname_matches_negative_refspec_item`, make the > function's purpose more intuitive, clarifying that it checks if a > reference name matches any negative refspec. The new name certainly reads way better, and the changes themselves look Patrick
