+1 to removing AUTHORS. I left some requests on the issue: * update some of the necessary docs * replace the AUTHORS content docs for the git shortlog -sne command * to do it for both pulp2 and pulp3.
https://github.com/pulp/pulp/pull/3393#issuecomment-378631901 On Wed, Apr 4, 2018 at 10:09 AM, Daniel Alley <dal...@redhat.com> wrote: > +0 > > On Wed, Apr 4, 2018 at 9:14 AM, Dana Walker <dawal...@redhat.com> wrote: > >> I like Austin's point about getting to specify contact info for a >> specific project as I have different emails for different types of >> contributions myself, but I don't want to have to update things in multiple >> places down the road when I can just keep track of my listed contact >> details in my github settings. The same goes for someone who wants privacy >> and doesn't want their email listed or someone who has changed their name. >> I created the AUTHORS file for pulp_file recently and had misgivings about >> whether each of these contributors would have chosen to have the info I put >> together as their representative contact info, especially since we squashed >> down to one name and email which involves making some assumptions. >> >> I also like Irina's points about redundancy with GitHub's tracking of >> contributors (though that's a very specific thing that excludes merge >> commits, and for instance, I have made contributions to pulp through PRs >> and my name is not listed there), and more importantly, inaccurate and out >> of date info when hardcoded. >> >> I'm +1 to this PR to remove AUTHORS files moving forward. >> >> --Dana >> >> >> Dana Walker >> >> Associate Software Engineer >> >> Red Hat >> >> <https://www.redhat.com> >> <https://red.ht/sig> >> >> On Wed, Apr 4, 2018 at 8:36 AM, Austin Macdonald <amacd...@redhat.com> >> wrote: >> >>> One reason to keep the AUTHORS file is that it allows contributors to >>> specify their contact info instead of just linking to their GitHub page, >>> were they might not have an email listed. >>> >>> I assume this discussion is restricted to pulp/pulp. The plugins are >>> free to handle however they prefer. >>> >>> Personally, I am fine either way. Remove +0, one less file. >>> >>> >>> On Wed, Apr 4, 2018, 05:22 Irina Gulina <igul...@redhat.com> wrote: >>> >>>> Hello Pulp Team, >>>> >>>> Recently, I have opened a PR to remove the AUTHORS file [1], and was >>>> suggested to bring it up here. >>>> Please let us/me know what you think about it. Any feedback will be >>>> appreciated. >>>> >>>> Happy Wednesday, >>>> Irina >>>> >>>> >>>> [1] https://github.com/pulp/pulp/pull/3393 >>>> >>>> _______________________________________________ >>>> Pulp-dev mailing list >>>> Pulp-dev@redhat.com >>>> https://www.redhat.com/mailman/listinfo/pulp-dev >>>> >>>> >>> _______________________________________________ >>> Pulp-dev mailing list >>> Pulp-dev@redhat.com >>> https://www.redhat.com/mailman/listinfo/pulp-dev >>> >>> >> >> _______________________________________________ >> Pulp-dev mailing list >> Pulp-dev@redhat.com >> https://www.redhat.com/mailman/listinfo/pulp-dev >> >> > > _______________________________________________ > Pulp-dev mailing list > Pulp-dev@redhat.com > https://www.redhat.com/mailman/listinfo/pulp-dev > >
_______________________________________________ Pulp-dev mailing list Pulp-dev@redhat.com https://www.redhat.com/mailman/listinfo/pulp-dev
