Skip to content

Conversation

@stevenc-stb
Copy link
Collaborator

No description provided.

Signed-off-by: stevenc-stb <steven@smarttalkbeacon.com>
@stevenc-stb stevenc-stb added this to the 3.1 milestone Oct 30, 2025
@stevenc-stb stevenc-stb added Profile:Hardware Hardware profile and related matters Profile:SupplyChain SupplyChain profile and related matters labels Oct 30, 2025
@bact bact added the documentation Improvements or additions to documentation label Oct 30, 2025
@bact
Copy link
Collaborator

bact commented Oct 30, 2025

If we can have a short summary of changes, it will make the review easier.

Signed-off-by: stevenc-stb <steven@smarttalkbeacon.com>
@stevenc-stb
Copy link
Collaborator Author

stevenc-stb commented Oct 30, 2025

If we can have a short summary of changes, it will make the review easier.

  • added the SupportRelationship to model.drawio
  • fixed fonts to Verdana size 10
  • fixed spaces in class rows example " + supportLevel : SupportType [0..1]" -> "+ supportLevel: SupportType[0..1]" 10+ places
  • fixed External properties restrictions -name: xsd:string[1]> "+ /Core/Element/name: xsd:string[1]"
  • fixed to show External properties restrictions + /Core/Element/name: xsd:string[1]
  • fixed an Core// Element to Core/ Element in build
  • change External properties restrictions to blue from red for background colour in SC + hardware .. core updates

Signed-off-by: stevenc-stb <steven@smarttalkbeacon.com>
@stevenc-stb
Copy link
Collaborator Author

  • add + put inLanguage in the right spot in the DatasetPackage.

@bact
Copy link
Collaborator

bact commented Oct 31, 2025

fixed External properties restrictions -name: xsd:string[1]> "+ /Core/Element/name: xsd:string[1]"

I guessed these are for Package, File, etc. - checked all instances of "name", looks good.

Three minor points on spacing:

In Hardware profile:

Screenshot 2568-10-31 at 20 59 35

the last two properties of Hardware class:

  • additionalInformationSpecification: /Core/Specification [0..*]
  • additionalInformation: /Core/DictionaryEntry [0..*]

there should be no space before [0..*]

In SupplyChain:

Screenshot 2568-10-31 at 20 53 24

the first property of State class:

  • /Core/Artifact/validUntilTime :DateTime [0]

  • there should be no space before :,

  • there should be one space after :,

  • there should be no space before [0]

These are more cosmetic and don't affect any semantic (we don't actually have these officially defined anyway) but would be nice to keep them consistent with others.

@bact
Copy link
Collaborator

bact commented Oct 31, 2025

Apart from those minor spacing consistency, all LGTM.

Btw, in the future you may also like to export the diagrams to PNGs and put them in https://github.com/spdx/spdx-3-model/tree/develop/images -- so the GitHub can show us the "image diffs" (GitHub's onion skin image review can be very helpful).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation Profile:Hardware Hardware profile and related matters Profile:SupplyChain SupplyChain profile and related matters

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants