7 Matching Annotations
- Dec 2020
-
www.npmjs.com www.npmjs.com
-
You can afford to make a proper PR to upstream.
-
The change would be useful to other people as-is.
-
The change is too consequential to be developed in situ.
-
Patches can be reviewed as part of your normal review process, forks probably can't
-
Get told in big red letters when the dependency changed and you need to check that your fix is still valid.
-
Keep your patches colocated with the code that depends on them.
-
Sometimes forks need extra build steps, e.g. with react-native for Android. Forget that noise.
Tags
- when to _
- pull request
- limited time: not right now (maybe later)
- advantages/merits/pros
- doing it right/properly
- forking (patching vs. forking)
- errors/warnings are helpful for development
- taking the time to do it right/properly
- proper
- review process
- patching vs. forking
- patching (patching vs. forking)
- helpful
- code organization: co-location
- too hard/complicated/non-trivial
Annotators
URL
-