Sebastian, this is awesome! I did have the same reaction as Alexios. I think we'll want to tweak the wording, to clarify the process -- e.g. that we've approved it to add, but that it'll still need the XML and test text file to be added. We could probably even point them to documentation about how to contribute those files themselves, and clarify that they can do so or else it'll be subject to whether / when another community member chooses to add them.
So we can figure out the right wording to use -- but I really like this as an added piece of automation and I think it'll help with communication around the process. Thanks for putting this together! Steve On Thu, Aug 25, 2022 at 2:57 AM Alexios Zavras <alexios.zav...@intel.com> wrote: > Nice automation work, Sebastian! > > I'm probably missing something, but... > The review and the labeling as "accepted" is happening on the issue where > someone submits a new license. > In order to have "the license's information added to the repository", we > also need a PR with the license XML, test file, etc. > > How is this handled? > I definitely remember cases in the past where licenses were accepted and > then we had to wait a long time for a PR (or Steve usually went ahead and > created one). > > -- zvr > > -----Original Message----- > From: Spdx-legal@lists.spdx.org <Spdx-legal@lists.spdx.org> On Behalf Of > Sebastian Crane > Sent: Thursday, 25 August, 2022 00:30 > To: SPDX Legal Mailing List <spdx-legal@lists.spdx.org> > Subject: Automatically responding to accepted additions to the SPDX > License List > > Dear all, > > I have good news! I've been able to create a CI workflow which responds to > accepted license requests on GitHub automatically, letting everyone know > that the license won't be visible on the website until the next release. > > Attached is a screenshot from a test repository of mine, where I've > simulated opening, labelling as accepted, then closing a 'New License > Request' issue. To prevent it from running repeatedly, the workflow adds a > 'finalised' label which it checks for before commenting. Let me know what > you think of the wording and we can get this running for real! > > Best wishes, > > Sebastian > > > > > > Intel Deutschland GmbH > Registered Address: Am Campeon 10, 85579 Neubiberg, Germany > Tel: +49 89 99 8853-0, www.intel.de <http://www.intel.de> > Managing Directors: Christin Eisenschmid, Sharon Heck, Tiffany Doon Silva > Chairperson of the Supervisory Board: Nicole Lau > Registered Office: Munich > Commercial Register: Amtsgericht Muenchen HRB 186928 > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#3228): https://lists.spdx.org/g/Spdx-legal/message/3228 Mute This Topic: https://lists.spdx.org/mt/93237695/21656 Group Owner: spdx-legal+ow...@lists.spdx.org Unsubscribe: https://lists.spdx.org/g/Spdx-legal/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
