feat: EC2 2023 8.2-Shear#121
feat: EC2 2023 8.2-Shear#121DanielGMorenaFhecor wants to merge 42 commits intofib-international:dev-ec2-2023from
Conversation
Co-authored-by: talledodiego <38036285+talledodiego@users.noreply.github.com>
…codes into ec_2004_crack_control
mortenengen
left a comment
mortenengen
left a comment
There was a problem hiding this comment.
Thanks for the great contribution 😃
I have merged dev and fixed the docstrings.
There is a significant number of lines that are not covered by tests. Please add tests so that we reach 100% coverage on the added code.
|
I have added tests to cover all 100% lines of code. Also, I have merged changes from |
talledodiego
left a comment
talledodiego
left a comment
There was a problem hiding this comment.
Thanks for this gigantic contribution. This is definitely a massive work!
I finished the review excluded the tests that I will check after the changes requested are performed on the other file.
talledodiego
left a comment
talledodiego
left a comment
There was a problem hiding this comment.
Thanks for this gigantic contribution. This is definitely a massive work!
I finished the review excluded the tests that I will check after the changes requested are performed on the other file.
talledodiego
moved this from Compliance review
to Changes requested
in EC2 2023 PR tracker
|
Thanks a lot, @talledodiego , for taking the time to review this PR, I really appreciate it, especially given the large number of lines to go through. I've addressed all your comments and applied the suggested changes. Please let me know if you spot anything else that needs adjustment! |
3 participants
I have completed the chapter '8.2 - Shear' from the EC2-2023.