ExifTool Forum

General => Other Discussion => Topic started by: MarshMello on February 10, 2024, 05:19:35 PM

Title: What makes you close all PRs?
Post by: MarshMello on February 10, 2024, 05:19:35 PM
This is more of a question directed to Phill but can be answered by anyone.

I was looking at the Exif Tool GitHub, after wondering if I can submit a PR to fix the issue I had a while ago. Saw that all PRs get closed (https://github.com/exiftool/exiftool/pulls?q=is%3Apr+is%3Aclosed), not merged but then you commit the code from a different name "exiftool".

I'm curious, why this approach?
Title: Re: What makes you close all PRs?
Post by: Phil Harvey on February 10, 2024, 05:30:30 PM
I merge all PR's by hand, and usually I make adjustments when I do this.

I'm not really familiar with best practices on github, but I have an "exiftool" user that owns the ExifTool code and does all of the updates via the command line, but I usually use my personal account when accessing via the web interface.

If you are an experienced github user and this seems odd to you, then it is probably because I'm not.  :P

- Phil
Title: Re: What makes you close all PRs?
Post by: MarshMello on February 10, 2024, 05:51:04 PM
Quote from: Phil Harvey on February 10, 2024, 05:30:30 PM...
- Phil

Makes sense,  thanks for the super quick response!