Skip to content

Conversation

venetrius
Copy link
Member

Copy link
Contributor

@psavidis psavidis left a comment

Choose a reason for hiding this comment

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

Good job to describe a very technical concept which is very specific.
In this regard, the description of the changes is indeed harder.

I've listed below some recommendations for changing the phrasing which was a bit confusing to me.

@venetrius
Copy link
Member Author

I've listed below some recommendations for changing the phrasing which was a bit confusing to me.
Thanks!
They all are great improvement, committed them.

@venetrius venetrius requested a review from psavidis October 4, 2024 11:55
@psavidis psavidis force-pushed the 4632-start-date-modify-operation branch from 74a2020 to 801adcb Compare October 4, 2024 12:11
@venetrius venetrius merged commit 0f236e5 into master Oct 4, 2024
1 check passed
@venetrius venetrius deleted the 4632-start-date-modify-operation branch October 4, 2024 12:16
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