ignore forks #1

michaelficarra opened this Issue Aug 29, 2011 · 5 comments


None yet
3 participants

See damianb/phar-util#1

You should probably ignore forks. Even better: submit the pull request to the forked repo instead if the spelling error also exists there.

damianb commented Aug 29, 2011

one thing worth noting is the potential existence of spelling errors in just the forks; the primary fork should be checked for the presence of that error before sending the PR to the main repo.

Yes, hence this section of my original request:

if the spelling error also exists there

damianb commented Aug 29, 2011

...Reading isn't really necessary anyways! Hmph!

(derp, my bad)


holdenk commented Aug 29, 2011

Yah thats a good idea, I'll add a check to see if its a fork and if so only check the root repo. I've turned off the script until I can get to this, but that won't be until after burning man.

holdenk was assigned Aug 29, 2011


holdenk commented Sep 14, 2013

Well this issue is ~2 years old and its finally fixed, so closing it for now :)

holdenk closed this Sep 14, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment