Difference between revisions of "Legal Team/Templatizing/ReviewXML"

Revision as of 19:52, 25 April 2016

Plan for review of XML license file

The files can be found here: https://github.com/spdx/license-list-XML

You will need to have a Github account to participate in the review. Let Jilayne know your Github account ID to be added to the repository team.

Key things to review before reviewing the XML files:

• SPDX License List Matching Guidelines: http://spdx.org/spdx-license-list/license-list-overview
• Explanation of the SPDX License List fields: http://spdx.org/spdx-license-list/license-list-overview (at bottom of page)
• Description of current proposal and XML tags: http://wiki.spdx.org/view/Legal_Team/Templatizing/tags-matching

Watch the following videos that Kris made:

• Review process demonstrated: https://www.youtube.com/watch?v=E_VoRQbDyEc
• Using Git (on Windows): https://www.youtube.com/watch?v=nsSjT4cmVEc -- why/who needs this as pertains to process?

Key things to look at during review:

1. identify which parts of the license template are important or unimportant for purposes of matching (as per the Matching Guidelines)
2. structural components - ensuring we have groups of clauses correctly identified (automated process takes care of most of this, but sanity check)
3. consider any changes that need to be made to the Matching Guidelines or License List Overview/description of fields

Process for Review

Labels created in Github: https://github.com/spdx/license-list-XML/labels

• if you find a license that has acknowledgment text, then label it with "has acknowledgement" - for example see:
• use ... define these??

• If you have approved the content, but otherwise approved, use "approved" label
• If the license is good to go, then click on the "merge pull request" and you're done!