Skip to content

Conversation

@jakubbortlik
Copy link
Collaborator

This is basically a duplicate of #485, but rebased onto main, in case you (@harrisoncramer) want to merge it directly without going through develop. I'm not sure which way you prefer, but I'll be mostly unavailable for the rest of the week and I don't want to keep the users waiting for the fix (and I think it would be a pity to just revert the jump-to-reviewer fix).

* fix: use new file name if old file name is empty (harrisoncramer#486)
* fix: swap `file_name` and `old_file_name` in reviewer data
* fix: jump to non-renamed files

---------

Co-authored-by: Jakub F. Bortlík <jakub.bortlik@proton.me>
@jakubbortlik jakubbortlik force-pushed the fix-internal-server-error branch from 7ad1c9b to aedf084 Compare April 3, 2025 10:54
@harrisoncramer harrisoncramer changed the base branch from main to develop April 9, 2025 13:46
@harrisoncramer harrisoncramer changed the base branch from develop to main April 9, 2025 13:46
@harrisoncramer
Copy link
Owner

harrisoncramer commented Apr 9, 2025

I don't want to merge something directly into main, I've merged #485 instead.

@jakubbortlik jakubbortlik deleted the fix-internal-server-error branch April 15, 2025 04:20
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.

2 participants