Skip to content

build: remove repo.spring.io from workflows#7340

Merged
jeremylong merged 1 commit intodependency-check:mainfrom
chadlwilson:patch-1
Jan 22, 2025
Merged

build: remove repo.spring.io from workflows#7340
jeremylong merged 1 commit intodependency-check:mainfrom
chadlwilson:patch-1

Conversation

@chadlwilson
Copy link
Collaborator

The FP workflows are failing due to this repo being unauthorised for access. Not sure how it was working before unless there was an (expired?) authentication token somewhere, as anonymous access I believe was blocked some time ago.

Do we need this here, or is Maven Central good enough?

The FP workflows are failing due to this repo being unauthorised for access. Not sure how it was working before unless there was an authentication token somewhere, as anonymous access I believe was blocked some time ago.

Do we need this here, or is Maven Central good enough?
@chadlwilson chadlwilson changed the title Remove repo.spring.io from workflows build: remove repo.spring.io from workflows Jan 22, 2025
Copy link
Collaborator

@jeremylong jeremylong left a comment

Choose a reason for hiding this comment

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

LGTM

@jeremylong
Copy link
Collaborator

The repo is unnecessary - I think I was trying to add as many alternatives as possible in case the FP came from another public repo. If this is no longer public we should remove it.

@jeremylong jeremylong merged commit cc0d5ed into dependency-check:main Jan 22, 2025
9 of 10 checks passed
@jeremylong jeremylong added this to the 12.0.2 milestone Jan 22, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 22, 2025
@chadlwilson chadlwilson deleted the patch-1 branch September 22, 2025 03:04
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.

2 participants

Comments