I have added a PR (https://github.com/apache/daffodil-vscode/pull/513) to add my co-worker Robert Strickland, a developer who has already contributed substantially to the Daffodil-VSCode repository. It is now in review for 2 others to sign off on.
Hopefully this is sufficient. On 3/22/23, 12:48 PM, "Steve Lawrence" <[email protected]> wrote: One difference is PR emails go to [email protected], so if people don't closely track that list they might not see that. Emails to [email protected] stand out a bit more (at least to me). Maybe that doesn't matter though, I don't have a strong opinion either way. On 2023-03-22 12:44 PM, Mike Beckerle wrote: > I suggest add/remove collaborators is always done as a distinct PR that > ONLY adds/removes collaborators. > > That eliminates a need for redundant emails to this list that are no > different than the PR itself. > > On Wed, Mar 22, 2023 at 12:41 PM Steve Lawrence <[email protected]> > wrote: > >> Since this only grants access to GitHub issues and wiki (is that true?), >> maybe we follow a similar policy that we use for Jira or Confluence? >> Which is users just ask on the dev list and provide their username, and >> then a comitter/PMC can grant the access. In this case granting the >> access is via a code change/PR instead of a config change. >> >> I'm not sure it needs to be any more complex than that, but I think it's >> good to require the requests be done publicly so others are aware when >> access changes are made. >> >> >> On 2023-03-22 12:30 PM, Shearer, Davin wrote: >>> There is a file called .asf.yaml that has a stanza where collaborators >> can be added (or removed). Since updates to .asf.yaml requires review by >> at least 2 other Apache members before merging into main, is that >> sufficient, or are there additional requirements for adding collaborators >> aside from the 2 green checks and the merge? Do we need any other rules >> beyond that? >>> >>> Thanks! >>> >>> Davin Shearer >>> Senior Principal Engineer >>> Concurrent Technologies Corporation (CTC) >>> 8539 Corridor Drive >>> Savage, MD 20763 >>> Office: (301) 543-3905 >>> [email protected]<mailto:[email protected]> >>> [cid110391*[email protected]] >>> >>> www.ctc.com<http://www.ctc.com/> >> [cid110391*[email protected]] < >> https://facebook.com/ConcurrentTechnologiesCorporation> >> [cid110391*[email protected]] < >> https://www.linkedin.com/company/concurrent-technologies-corporation> >> [cid110391*[email protected]] < >> https://twitter.com/ConcurrentTech> >> [cid110391*[email protected]] < >> https://www.youtube.com/c/ConcurrentTechnologiesCorporation> >>> >>> >>> ----------------------------------------------------------------- >>> This message and any files transmitted within are intended >>> solely for the addressee or its representative and may contain >>> company proprietary information. If you are not the intended >>> recipient, notify the sender immediately and delete this >>> message. Publication, reproduction, forwarding, or content >>> disclosure is prohibited without the consent of the original >>> sender and may be unlawful. >>> >>> Concurrent Technologies Corporation and its Affiliates. >>> www.ctc.com 1-800-282-4392 >>> ----------------------------------------------------------------- >> >> > ----------------------------------------------------------------- This message and any files transmitted within are intended solely for the addressee or its representative and may contain company proprietary information. If you are not the intended recipient, notify the sender immediately and delete this message. Publication, reproduction, forwarding, or content disclosure is prohibited without the consent of the original sender and may be unlawful. Concurrent Technologies Corporation and its Affiliates. www.ctc.com 1-800-282-4392 -----------------------------------------------------------------
