feat: implement skip_contract_check kwarg

[charles-cooper]

What I did

Implement #2550

How I did it

Pretty straightforward, thread the argument through typechecker and codegen

How to verify it

See tests

Description for the changelog

Add omit_contract_check kwarg as an external call option

Cute Animal Picture

[codecov-commenter]

Codecov Report

Merging #2551 (7ab7ab1) into master (2b96fd8) will increase coverage by 0.01%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master    #2551      +/-   ##
==========================================
+ Coverage   85.65%   85.66%   +0.01%     
==========================================
  Files          90       90              
  Lines        9027     9036       +9     
  Branches     2324     2328       +4     
==========================================
+ Hits         7732     7741       +9     
  Misses        814      814              
  Partials      481      481              

Impacted Files	Coverage Δ
vyper/__init__.py	81.81% <ø> (ø)
vyper/semantics/types/function.py	84.93% <71.42%> (+0.32%)	⬆️
vyper/old_codegen/external_call.py	94.64% <92.85%> (+0.30%)	⬆️
vyper/evm/opcodes.py	100.00% <0.00%> (ø)
vyper/compiler/utils.py	100.00% <0.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2b96fd8...7ab7ab1. Read the comment docs.

[charles-cooper]

@transmissions11 up for review if you have any feedback before this gets merged

[transmissions11]

omit_contract_check is a bit verbose but otherwise this looks great 👍

[charles-cooper]

omit_contract_check is a bit verbose but otherwise this looks great +1

@transmissions11 any alternatives? open to suggestions

[transmissions11]

no_code_check?

[charles-cooper]

I think we are going to go with skip_contract_check. (contract instead of code because sometimes it's a check on returndatasize instead of extcodesize).