Skip to content

Conversation

sylane
Copy link
Contributor

@sylane sylane commented Feb 20, 2025

The file grisp_connect_log_server.erl was renamed grisp_connect_log.erl and a lot of the code remained unchanged, but git is not recognising the rename because there is too much differences (all the gen_server stuff got removed).

Copy link
Member

@ziopio ziopio left a comment

Choose a reason for hiding this comment

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

Looks Good

@sylane sylane force-pushed the sylane/log-pull branch 2 times, most recently from 3afd8b2 to 4a8e375 Compare February 21, 2025 18:50
@ziopio
Copy link
Member

ziopio commented Feb 24, 2025

You should have left the WIP commit and just added another one, now I am using the compare buttons to know what changed from my partial review.
It is cool that Github helps with this but in this case @sylane there was no need to rewrite the commit.
;)

@sylane
Copy link
Contributor Author

sylane commented Feb 24, 2025

You should have left the WIP commit and just added another one, now I am using the compare buttons to know what changed from my partial review. It is cool that Github helps with this but in this case @sylane there was no need to rewrite the commit. ;)

I rebased on top of the changes of the exponential backoff PR, so there is not much I could do, could I ?
This is the problem with multiple inter-dependent PR...

@ziopio
Copy link
Member

ziopio commented Feb 24, 2025

Maybe I got confused with another PR, yeah do not worry.

@sylane sylane force-pushed the sylane/add-exp-backoff branch from ba0f06f to ee158fc Compare February 24, 2025 16:49
Base automatically changed from sylane/add-exp-backoff to main February 24, 2025 16:59
@ziopio ziopio merged commit 977fd66 into main Feb 26, 2025
1 check 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.

3 participants