Martino Sorbaro
Martino Sorbaro
Thank you for catching the missing periods and for taking the time to submit this PR. I have to point out that the first change would render as `git restore...
I've updated it myself. I will not merge for a while because we are waiting for a larger merge which I don't want to conflict with this. Thank you.
Thanks - I have no objections to this but I also don't know much about this topic and whether the default has changed or git still gives the `hint` to...
The first part can be read as using assert for input validation, which we have all agreed is not right, but could also be interpreted as testing, if we assume...
Answering as a member of the CAC, but not representing the rest of the CAC. I can agree that one should not use assert as input validation (although not all...
I'll answer again to @dhvalden's points: 1. _Why would we want to teach tests if we haven't learned how to raise errors correctly first?_ As I said, I agree that...
Ok 🤷. I don't want to insist further and my opinion is as good as anyone's, so I'm happy to support what the community suggests.
I'm thinking of merging this as soon as we have an example of CFF either in the episode or as a separate file for the whole lesson. However, this gave...
@erinmgraham shall we merge this for now?
I think this was done on purpose. It says "you can forget..." but then the callout says "...but don't actually", and it's a callout because this is more of a...