Green-Software-Foundation / sci

A specification that describes how to calculate a carbon intensity for software applications.
Other
263 stars 54 forks source link

Add orc-id for CLJ to file #84

Closed Sealjay closed 3 years ago

Sealjay commented 3 years ago

Inspired by @atg-abhishek - not something I was familiar with before!

jawache commented 3 years ago

Is this document automatically generated from GH contributions? I'm a little nervous about this in general since the spec is from everyone in the foundation. i.e. NOT raising an objection to a PR is still contributing to the spec.

Sealjay commented 3 years ago

Is this document automatically generated from GH contributions? I'm a little nervous about this in general since the spec is from everyone in the foundation. i.e. NOT raising an objection to a PR is still contributing to the spec.

nope, this file was manually created by @atg-abhishek - I believe he was adding everyone within the foundation to it, and asked for others on the call to reach out if they noticed anyone missing. this PR is just to add my ID to the file that's there

I'm not able to merge by the way as I don't have write access - so I don't know if we can do this through the RNA process, as it seems a bit minor to raise on a call

Sealjay commented 3 years ago

More info @jawache - http://bit.ly/across-workshop-community-pitch - This is related to a bigger issue, around how to identify contributors - understanding explicitly that source control is not the full picture of open source, as it excludes governance, decision making, and consensus as you mentioned. https://citation-file-format.github.io/ - this file is one approach to recognize contributors without requiring people to update the repo

atg-abhishek commented 3 years ago

@Sealjay I've made a discussion based on your comment above so that it doesn't get lost as we close out this PR.

Sealjay commented 3 years ago

Are we ok to merge, or is there some kind of RNA approval needed @atg-abhishek ?

atg-abhishek commented 3 years ago

I'll merge this since we have the discussion captured separately.