-
-
Notifications
You must be signed in to change notification settings - Fork 175
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FR - Translation "Resource Hints" Chapter #652
FR - Translation "Resource Hints" Chapter #652
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thansk @borisschapira - spotted a few things for you to look at.
Co-Authored-By: Barry Pollard <barry_pollard@hotmail.com>
Co-Authored-By: Barry Pollard <barry_pollard@hotmail.com>
Co-Authored-By: Barry Pollard <barry_pollard@hotmail.com>
Thanks a lot @bazzadp. As always, you provide great feedback. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here we are, a quick review for typos and some questions :)
Co-Authored-By: Barry Pollard <barry_pollard@hotmail.com>
Co-Authored-By: Barry Pollard <barry_pollard@hotmail.com>
I've fixed a lot of issues (thanks!) and tried to answer everything else. For some question, I know there is not "good" answer so tell us if the answer I provided looks good, or if you think we need a second (or third) opinion. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good to merge from my side if @nico3333fr is happy.
@bazzadp I'm always happy :) (except when I'm sad, for sure) To be serious, this is ok for me :) Good job guys 👍 |
Thanks both! Merging away... |
Makes progress on #539