Hacker News
new
|
past
|
comments
|
ask
|
show
|
jobs
|
submit
login
xboxnolifes
on Dec 12, 2021
|
parent
|
context
|
favorite
| on:
We check our node_modules folder into source contr...
> have every commit in master
Yes, but a PR commit is not in master.
pjerem
on Dec 12, 2021
|
next
[–]
I see you, little squasher :)
ng12
on Dec 12, 2021
|
prev
[–]
But then you lose the benefit. Hiding node_modules during code review is the easy part.
Consider applying for YC's Fall 2025 batch! Applications are open till Aug 4
Guidelines
|
FAQ
|
Lists
|
API
|
Security
|
Legal
|
Apply to YC
|
Contact
Search:
Yes, but a PR commit is not in master.