github-csv-tools icon indicating copy to clipboard operation
github-csv-tools copied to clipboard

Support the import of pull requests? Our fork supports some functionality

Open jaylenw opened this issue 2 years ago • 5 comments

Hello, is there any interests to support the importation of Pull Requests? We are aware there are some "gotchas" when trying to import PRs with GitHub's API as certain edge cases are not supported. However, we have modified our fork of the code to support some cases of importing PRs. We have added documentation to our fork in the README.md file. If there is interest, please checkout our fork with the code that supports importing PRs here and if all is well let us know to open a PR to this repository. We are willing to make small changes to our fork so that a PR can be approved and merged into this repository but we can not spend much more time on our fork.

Thanks!

Branch with our changes: https://github.com/CityOfLosAngeles/github-csv-tools/tree/import-pull-requests Diff with our changes as of now: https://github.com/gavinr/github-csv-tools/compare/master...CityOfLosAngeles:import-pull-requests.

jaylenw avatar Apr 11 '22 23:04 jaylenw

Thanks a lot for the post. I really appreciate you posting the proposed code update. I'd like to leave this open to see if others chime in with the need for this feature.

gavinr avatar Apr 11 '22 23:04 gavinr

Hello @gavinr , no problem, your welcome. Thanks for your work on this tool!

jaylenw avatar Apr 12 '22 00:04 jaylenw

Thanks for the proposed code changes! I'd love to see this feature as well.

I just tried it with etcd.io/etcd (via githubCsvTools -p --exportAttributes=number,title,labels,state,assignees,milestone,comments,created_at,updated_at,closed_at), and it works well!

94rain avatar Oct 18 '22 05:10 94rain

Not to necro-post, but I want to see this as well.

DavidGriffith avatar Sep 18 '23 04:09 DavidGriffith

Hello @gavinr any updated decision on us opening a pull request to support the proposed functionality? Thanks.

jaylenw avatar Dec 05 '23 18:12 jaylenw