Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Issue 612: Warn if a Disk instance is not also a StorageMedium #620

Merged
merged 2 commits into from
Sep 11, 2024
Merged

Issue 612: Warn if a Disk instance is not also a StorageMedium #620

merged 2 commits into from
Sep 11, 2024

Conversation

ajnelson-nist
Copy link
Contributor

@ajnelson-nist ajnelson-nist commented Jul 26, 2024 •
edited
Loading

This Pull Request resolves all backwards-compatible requirements of Issue #612 .

Test cases omitted from ontology's tests because CASE example data are known to use uco-observable:Disk (currently in the "Partitions" example).

Coordination

  • Pull Request is against correct branch
  • Pull Request is in, or reverted to, Draft status before Solutions Approval vote has passed
  • CI passes in UCO feature branch against develop
  • CI passes in UCO current unstable branch (e7400c1)
  • CI passes in CASE current unstable branch tracking UCO's unstable as submodule (6f63009)
  • Impact on SHACL validation reviewed for CASE-Corpora
  • Impact on SHACL validation remediated for CASE-Corpora (N/A)
  • Impact on SHACL validation reviewed for CASE-Examples
  • Impact on SHACL validation remediated for CASE-Examples
  • Impact on SHACL validation reviewed for casework.github.io
  • Impact on SHACL validation remediated for casework.github.io
  • Milestone linked
  • Solutions Approval vote logged on corresponding Issue (once logged, can be taken out of Draft PR status)

@ajnelson-nist
Warn if a Disk instance is not also a Device
03a3d3c 
Test cases omitted from ontology's tests because CASE example data are
known to use `uco-observable:Disk`.

No effects were observed on Make-managed files.

References:
* #612

Signed-off-by: Alex Nelson <[email protected]>
@ajnelson-nist ajnelson-nist added this to the UCO 1.4.0 milestone Jul 26, 2024
@ajnelson-nist ajnelson-nist linked an issue Jul 26, 2024 that may be closed by this pull request
Should Disk be a subclass of Device? #612
Closed
13 tasks
@ajnelson-nist ajnelson-nist changed the title Issue 612: Warn if a Disk instance is not also a Device Issue 612: Warn if a Disk instance is not also a StorageMedium Jul 29, 2024
@ajnelson-nist
Warn if a Disk instance is not also a StorageMedium
a271750 
No effects were observed on Make-managed files.

References:
* #612

Signed-off-by: Alex Nelson <[email protected]>
@ajnelson-nist ajnelson-nist mentioned this pull request Aug 28, 2024
Closed
13 tasks
@ajnelson-nist
Copy link
Contributor Author

This PR is ready for review and merge.

@ajnelson-nist ajnelson-nist marked this pull request as ready for review September 11, 2024 13:36
@plbt5 plbt5 merged commit 178efd0 into develop Sep 11, 2024
1 check passed
@plbt5 plbt5 deleted the Feature-Issue-612 branch September 11, 2024 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@plbt5 plbt5 plbt5 approved these changes

Assignees
No one assigned
Labels
Change Proposal
Projects
None yet
Milestone
UCO 1.4.0
Development

Successfully merging this pull request may close these issues.

Should Disk be a subclass of Device?
2 participants
@ajnelson-nist @plbt5