Fix clippy on Nightly

[dvtkrlbs]

This PR just fixes clippy

It changes the following:

• It changes match blocks that returns booleans with two arms and if let blocks to the matches! macro

Ok i accidently tried this with nightly clippy and fixed a lint error for it. So this pr could be for nothing. But i can make it a draft PR and we can revisit it after this lint comes to the stable (if it ever does).

[codecov]

Codecov Report

Merging #668 into master will increase coverage by 0.02%.
The diff coverage is 44.44%.

@@            Coverage Diff             @@
##           master     #668      +/-   ##
==========================================
+ Coverage   71.76%   71.78%   +0.02%     
==========================================
  Files         199      199              
  Lines       13953    13943      -10     
==========================================
- Hits        10013    10009       -4     
+ Misses       3940     3934       -6     

Impacted Files	Coverage Δ
boa/src/builtins/string/mod.rs	50.72% <0.00%> (-0.12%)	⬇️
boa/src/syntax/lexer/mod.rs	66.00% <0.00%> (-0.67%)	⬇️
boa/src/environment/lexical_environment.rs	75.00% <37.50%> (-0.50%)	⬇️
...tax/parser/expression/assignment/exponentiation.rs	85.00% <66.66%> (+4.04%)	⬆️
boa/src/environment/function_environment_record.rs	27.65% <100.00%> (+0.57%)	⬆️
boa/src/syntax/parser/expression/assignment/mod.rs	63.52% <100.00%> (+1.46%)	⬆️
boa/src/value/mod.rs	69.85% <100.00%> (+0.09%)	⬆️

---

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 ad7cd4c...0d50c41. Read the comment docs.

[Razican]

Also why clippy is not enabled on CI?

Isn't it? I thought we had an specific test, that's odd.

[dvtkrlbs]

Oh there is. How come it was not failing for this lint ? Ok i am dumb i was running clippy on nightly and never tried it on stable.

[Lan2u]

Approved but only once this is stable.

[Razican]

I'm happy to merge this as-is. The matches!() macro is already stable, it's the lint the only thing that is unstable, so merging this would future-proof the project. What do everyone else think?

[Lan2u]

I'm happy to merge this as-is. The matches!() macro is already stable, it's the lint the only thing that is unstable, so merging this would future-proof the project. What do everyone else think?

If @dvtkrlbs can resolve the merge conflict then I'm happy to merge this