fix: lint during the landing process

[codebytere]

Closes #427.

Runs make lint after the patch has been downloaded and applied, and optionally stage and amend updated files if lint fails.

[codecov]

Codecov Report

Merging #435 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #435      +/-   ##
==========================================
- Coverage   82.58%   82.56%   -0.02%     
==========================================
  Files          34       34              
  Lines        1665     1658       -7     
==========================================
- Hits         1375     1369       -6     
+ Misses        290      289       -1     

Impacted Files	Coverage Δ
lib/ci/ci_type_parser.js	81.35% <0.00%> (-0.47%)	⬇️
lib/pr_checker.js	95.49% <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 fd566e2...9c846de. Read the comment docs.

[richardlau]

Closes #427.

Runs make lint after the patch has been downloaded and applied, and optionally stage and amend updated files if lint fails.

Isn’t that going to fail on Windows?

[codebytere]

@richardlau you might be right - my esoteric windows knowledge is admittedly weaker 😅

[targos]

Does git node land currently work on windows ?

[codebytere]

We could also only run make lint on linux/macOS potentially

[BridgeAR]

We might want add some tests for these changes.

[codebytere]

I don't know we have tests for any of the actual patch downloading code right now - @joyeecheung what do you think?

[codebytere]

@lundibundi i cleaned up the flow a bit - i think this is a bit better for users now