Skip to content

fix: remove non-reproducible git entries#5079

Merged
jeremylong merged 1 commit into
dependency-check:mainfrom
hboutemy:main
Nov 21, 2022
Merged

fix: remove non-reproducible git entries#5079
jeremylong merged 1 commit into
dependency-check:mainfrom
hboutemy:main

Conversation

@hboutemy

Copy link
Copy Markdown
Contributor

Fixes Issue

fixes #5026

Description of Change

ignore 2 non-reproducible git properties

Have test cases been added to cover the new functionality?

no, but tested with a build

@aikebah aikebah left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks sensible to me to exclude these for reproducibility and from the looks of it git.commit.count was not a valid property key.

@jeremylong jeremylong changed the title fix non-reproducible git entries fix: remove non-reproducible git entries Nov 21, 2022
@jeremylong jeremylong merged commit c2d7108 into dependency-check:main Nov 21, 2022
@jeremylong jeremylong added this to the 7.4.0 milestone Nov 21, 2022
@hboutemy hboutemy deleted the main branch November 23, 2022 07:11
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Dec 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improving reproducible build

3 participants