Skip to content

Conversation

@rajesh-jonnalagadda
Copy link

@rajesh-jonnalagadda rajesh-jonnalagadda commented Nov 11, 2024

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Nov 11, 2024
@github-actions github-actions bot added modifies/go Pull requests that update Go code modifies/templates This PR modifies the template files labels Nov 11, 2024
@rajesh-jonnalagadda rajesh-jonnalagadda force-pushed the feat-32257-add-comments-unchanged-lines-and-show branch 2 times, most recently from e59800b to ed80303 Compare November 11, 2024 02:46
@rajesh-jonnalagadda rajesh-jonnalagadda force-pushed the feat-32257-add-comments-unchanged-lines-and-show branch from ed80303 to abd1d2f Compare November 11, 2024 03:34
@rajesh-jonnalagadda rajesh-jonnalagadda marked this pull request as draft November 11, 2024 16:59
@rajesh-jonnalagadda rajesh-jonnalagadda force-pushed the feat-32257-add-comments-unchanged-lines-and-show branch from 8c34f45 to 7c39ad2 Compare November 11, 2024 17:27
@rajesh-jonnalagadda rajesh-jonnalagadda force-pushed the feat-32257-add-comments-unchanged-lines-and-show branch from 3fbf96f to ccd797a Compare November 11, 2024 17:50
@rajesh-jonnalagadda rajesh-jonnalagadda marked this pull request as ready for review November 11, 2024 18:29
@rajesh-jonnalagadda rajesh-jonnalagadda marked this pull request as draft November 11, 2024 19:19
@rajesh-jonnalagadda rajesh-jonnalagadda marked this pull request as ready for review November 11, 2024 23:56
@rajesh-jonnalagadda
Copy link
Author

@lunny Can you please review

@rajesh-jonnalagadda
Copy link
Author

@lunny Could you please review my changes? If the PR isn't relevant, feel free to close it.

@lunny
Copy link
Member

lunny commented Nov 13, 2024

@lunny Could you please review my changes? If the PR isn't relevant, feel free to close it.

I will review it in the following week.

@rajesh-jonnalagadda
Copy link
Author

@lunny Do we need these changes?

@lunny
Copy link
Member

lunny commented Jan 27, 2025

@lunny Do we need these changes?

Hi, #32471 (comment) hasn't been updated and there are many changes and conflicts.

@rajesh-jonnalagadda
Copy link
Author

rajesh-jonnalagadda commented Mar 2, 2025

@lunny resolved the conflicts and removed the j-query part. can you please review

@rajesh-jonnalagadda rajesh-jonnalagadda force-pushed the feat-32257-add-comments-unchanged-lines-and-show branch from 03493b9 to ebad7a3 Compare March 2, 2025 10:46
@rajesh-jonnalagadda
Copy link
Author

@lunny can please review the pr

@rajesh-jonnalagadda
Copy link
Author

@lunny can please review the pr

@lunny ^^

@rajesh-jonnalagadda
Copy link
Author

@lunny can you please review?

@rajesh-jonnalagadda
Copy link
Author

@lunny can you please review?

@lunny do we need these changes.

@rajesh-jonnalagadda
Copy link
Author

@lunny can you please review?

@lunny do we need these changes.

@lunny ^^ Can you please review

@lunny
Copy link
Member

lunny commented Oct 19, 2025

Replaced by #35662

@lunny lunny closed this Oct 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🙋 Bounty claim lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. modifies/frontend modifies/go Pull requests that update Go code modifies/templates This PR modifies the template files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants