Skip to content

Conversation

@hunshcn
Copy link
Contributor

@hunshcn hunshcn commented Jul 29, 2024

This is a code error (from typo).

@hunshcn
Copy link
Contributor Author

hunshcn commented Jul 29, 2024

Reported by @Whoaa512 .

I didn't modify changelog and test because although I believe this is a mistake, I didn't think of how to trigger this problem.

Did you trigger this error in some scenes? @Whoaa512

@Whoaa512
Copy link
Contributor

No, I just noticed because I was referencing at rules_python while building a new language plugin.

@aignas
Copy link
Collaborator

aignas commented Dec 3, 2024

@hunshcn, is this still needed? If so, could you please rebase?

@hunshcn hunshcn marked this pull request as ready for review December 4, 2024 03:25
@hunshcn hunshcn requested a review from aignas as a code owner December 4, 2024 03:25
@hunshcn
Copy link
Contributor Author

hunshcn commented Dec 4, 2024

Should I modify changelog? There should be no user-oriented modification.

@aignas
Copy link
Collaborator

aignas commented Dec 4, 2024

I think no need to change the changelog, FYI @dougthor42.

Copy link
Collaborator

@dougthor42 dougthor42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed, no update to changelog needed.

@aignas aignas added this pull request to the merge queue Dec 4, 2024
Merged via the queue into bazel-contrib:main with commit 60da4a3 Dec 4, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants