Move spdx extensions to common utils and refactor SPDX 2.2 by pragnya17 · Pull Request #998 · microsoft/sbom-tool (original) (raw)

Skip to content

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@pragnya17

As part of a larger effort to refactor the SPDX extensions in SPDX 2.2 and SPDX 3.0, this PR aims to move the non-extension method to a common utility class. It also removes the duplicates from the SPDX 2.2 project and utilizes the common utility methods. Future PRs will target refactoring for SPDX 3.0.

ppandrate added 2 commits

April 2, 2025 14:20

DaveTryon

DaveTryon

ppandrate and others added 4 commits

April 3, 2025 11:40

@pragnya17

@pragnya17

@pragnya17 pragnya17 changed the titleMove spdx extensions to common utils Move spdx extensions to common utils and refactor SPDX 2.2

Apr 3, 2025

ZhengHong-Tan

DaveTryon

DaveTryon

@pragnya17

1 similar comment

@pragnya17

@azure-pipelines

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

@pragnya17

@pragnya17

DaveTryon

DaveTryon

DaveTryon

DaveTryon

DaveTryon

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now that we're using constants for the comparison, we might be able to remove some tests

@pragnya17

DaveTryon

@pragnya17 pragnya17 deleted the ppandrate_moveSpdxExtensionsToCommonUtils branch

April 4, 2025 00:06

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})