[0.24] Ignore missing comma in parameter list parse error for contract updates

[turbolent]

Closes #1710

---

• Targeted PR against master branch
• Linked to Github issue with discussion and accepted design OR link to spec that describes this work
• Code follows the standards mentioned here
• Updated relevant documentation
• Re-reviewed Files changed in the Github PR explorer
• Added appropriate labels

[github-actions]

Cadence Benchstat comparison

This branch with compared with the base branch onflow:master commit f4bc9b2
The command for i in {1..N}; do go test ./... -run=XXX -bench=. -benchmem -shuffle=on; done was used.
Bench tests were run a total of 7 times on each branch.

Results

	old.txt	new.txt
	time/op		delta
CheckContractInterfaceFungibleTokenConformance-2	145µs ± 3%	149µs ± 7%	~	(p=0.318 n=7+7)
ContractInterfaceFungibleToken-2	44.0µs ± 2%	45.0µs ±10%	~	(p=0.535 n=7+7)
InterpretRecursionFib-2	2.94ms ± 4%	2.93ms ± 6%	~	(p=1.000 n=7+7)
NewInterpreter/new_interpreter-2	1.16µs ± 2%	1.16µs ± 3%	~	(p=0.753 n=7+6)
NewInterpreter/new_sub-interpreter-2	2.46µs ± 4%	2.50µs ± 3%	~	(p=0.209 n=7+7)
ParseArray-2	8.98ms ± 4%	9.15ms ± 4%	~	(p=0.181 n=7+6)
ParseDeploy/byte_array-2	13.7ms ± 3%	13.8ms ± 5%	~	(p=0.535 n=7+7)
ParseDeploy/decode_hex-2	1.21ms ± 4%	1.24ms ± 5%	~	(p=0.209 n=7+7)
ParseFungibleToken/With_memory_metering-2	217µs ± 3%	226µs ±11%	~	(p=0.165 n=7+7)
ParseFungibleToken/Without_memory_metering-2	167µs ± 6%	170µs ± 5%	~	(p=0.628 n=7+6)
ParseInfix-2	7.91µs ± 4%	7.91µs ± 3%	~	(p=1.000 n=7+6)
QualifiedIdentifierCreation/One_level-2	2.45ns ± 3%	2.45ns ± 4%	~	(p=0.836 n=7+6)
QualifiedIdentifierCreation/Three_levels-2	145ns ± 3%	153ns ± 8%	+5.65%	(p=0.001 n=7+7)
RuntimeFungibleTokenTransfer-2	1.19ms ±22%	1.26ms ±29%	~	(p=0.259 n=7+7)
RuntimeResourceDictionaryValues-2	6.76ms ± 5%	6.81ms ± 6%	~	(p=0.620 n=7+7)
Transfer-2	92.4ns ± 3%	95.4ns ± 5%	~	(p=0.165 n=7+7)
	alloc/op		delta
CheckContractInterfaceFungibleTokenConformance-2	66.3kB ± 0%	66.3kB ± 0%	~	(p=1.000 n=7+7)
ContractInterfaceFungibleToken-2	26.7kB ± 0%	26.7kB ± 0%	~	(all equal)
InterpretRecursionFib-2	1.25MB ± 0%	1.25MB ± 0%	~	(p=0.141 n=5+7)
NewInterpreter/new_interpreter-2	872B ± 0%	872B ± 0%	~	(all equal)
NewInterpreter/new_sub-interpreter-2	1.38kB ± 0%	1.38kB ± 0%	~	(all equal)
ParseArray-2	2.80MB ± 2%	2.81MB ± 2%	~	(p=0.535 n=7+7)
ParseDeploy/byte_array-2	4.45MB ± 3%	4.48MB ± 3%	~	(p=0.165 n=7+7)
ParseDeploy/decode_hex-2	214kB ± 0%	214kB ± 0%	~	(p=0.469 n=7+7)
ParseFungibleToken/With_memory_metering-2	36.3kB ± 0%	36.3kB ± 0%	~	(p=0.611 n=7+7)
ParseFungibleToken/Without_memory_metering-2	36.3kB ± 0%	36.3kB ± 0%	~	(p=0.793 n=7+7)
ParseInfix-2	2.17kB ± 0%	2.17kB ± 0%	~	(p=0.179 n=7+7)
QualifiedIdentifierCreation/One_level-2	0.00B	0.00B	~	(all equal)
QualifiedIdentifierCreation/Three_levels-2	64.0B ± 0%	64.0B ± 0%	~	(all equal)
RuntimeFungibleTokenTransfer-2	237kB ± 1%	236kB ± 0%	~	(p=0.620 n=7+7)
RuntimeResourceDictionaryValues-2	2.25MB ± 0%	2.25MB ± 0%	~	(p=0.902 n=7+7)
Transfer-2	48.0B ± 0%	48.0B ± 0%	~	(all equal)
	allocs/op		delta
CheckContractInterfaceFungibleTokenConformance-2	1.07k ± 0%	1.07k ± 0%	~	(all equal)
ContractInterfaceFungibleToken-2	460 ± 0%	460 ± 0%	~	(all equal)
InterpretRecursionFib-2	23.8k ± 0%	23.8k ± 0%	~	(all equal)
NewInterpreter/new_interpreter-2	12.0 ± 0%	12.0 ± 0%	~	(all equal)
NewInterpreter/new_sub-interpreter-2	40.0 ± 0%	40.0 ± 0%	~	(all equal)
ParseArray-2	70.0k ± 0%	70.0k ± 0%	~	(p=0.592 n=7+7)
ParseDeploy/byte_array-2	105k ± 0%	105k ± 0%	~	(p=0.592 n=7+7)
ParseDeploy/decode_hex-2	78.0 ± 0%	78.0 ± 0%	~	(all equal)
ParseFungibleToken/With_memory_metering-2	1.06k ± 0%	1.06k ± 0%	~	(all equal)
ParseFungibleToken/Without_memory_metering-2	1.06k ± 0%	1.06k ± 0%	~	(all equal)
ParseInfix-2	66.0 ± 0%	66.0 ± 0%	~	(all equal)
QualifiedIdentifierCreation/One_level-2	0.00	0.00	~	(all equal)
QualifiedIdentifierCreation/Three_levels-2	2.00 ± 0%	2.00 ± 0%	~	(all equal)
RuntimeFungibleTokenTransfer-2	4.52k ± 0%	4.52k ± 0%	~	(p=1.000 n=7+7)
RuntimeResourceDictionaryValues-2	37.6k ± 0%	37.6k ± 0%	~	(p=1.000 n=7+7)
Transfer-2	1.00 ± 0%	1.00 ± 0%	~	(all equal)

[SupunS]

LGTM!

[codecov]

Codecov Report

Merging #1711 (bb21268) into v0.24 (858865b) will decrease coverage by 0.01%.
The diff coverage is 65.51%.

@@            Coverage Diff             @@
##            v0.24    #1711      +/-   ##
==========================================
- Coverage   77.00%   76.99%   -0.02%     
==========================================
  Files         292      292              
  Lines       61187    61211      +24     
==========================================
+ Hits        47117    47129      +12     
- Misses      12459    12469      +10     
- Partials     1611     1613       +2     

Flag	Coverage Δ
unittests	76.99% <65.51%> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
runtime/parser2/errors.go	19.17% <12.50%> (-1.72%)	⬇️
runtime/runtime.go	86.47% <83.33%> (-0.04%)	⬇️
runtime/parser2/function.go	73.63% <100.00%> (+0.39%)	⬆️
runtime/parser2/parser.go	91.18% <0.00%> (-0.76%)	⬇️

---

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 858865b...bb21268. Read the comment docs.