Skip to content

Conversation

@pkalsi97
Copy link
Contributor

@pkalsi97 pkalsi97 commented Nov 15, 2025

Enables the forbidigo and tagalign linters and fix violations.

Issue: #5486

Copy link
Contributor

@squakez squakez 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!

@pkalsi97 pkalsi97 requested a review from squakez November 15, 2025 18:35
@pkalsi97
Copy link
Contributor Author

@squakez Resolved conflict.

@github-actions
Copy link
Contributor

⚠️ Unit test coverage report - coverage decreased from 51.5% to 51.4% (-0.1%)

@pkalsi97
Copy link
Contributor Author

pkalsi97 commented Nov 18, 2025

@squakez Do you plan to merger this later ? just asking, so I can avoid stacking conflicts in the next PR.

@squakez
Copy link
Contributor

squakez commented Nov 18, 2025

I cannot merge due to conflicts. As soon as you rebase and clear the conflicts I will be able to merge it.

@pkalsi97 pkalsi97 force-pushed the ci/enable-forbidigo-tagalign branch from 11b4a0a to f266094 Compare November 18, 2025 08:59
@pkalsi97
Copy link
Contributor Author

@squakez my bad, I missed the conflict earlier. It's resolved now.

@pkalsi97
Copy link
Contributor Author

@squakez just confirming, do you still see any conflicts ? The UI is not showing any, even previously it was not showing (not sure why).

@gansheer gansheer merged commit f07454a into apache:main Nov 19, 2025
10 checks passed
@pkalsi97 pkalsi97 deleted the ci/enable-forbidigo-tagalign branch November 19, 2025 08:11
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.

3 participants