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.

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

Update maven central repo URL #6754

Merged
merged 1 commit into from
Nov 28, 2023
Merged

Conversation

mbien
Copy link
Member

@mbien mbien commented Nov 27, 2023

  • switch everything to repo.maven.apache.org
  • which is the default used by maven
  • domain is managed by ASF which should make it future proof since it can redirect to other places

 - switch everything to repo.maven.apache.org
 - which is the default used by maven
 - domain is managed by ASF which should make it future proof since
   it can be redirected
@mbien mbien added Code cleanup Maven [ci] enable "build tools" tests ci:all-tests [ci] enable all tests labels Nov 27, 2023
@mbien mbien added this to the NB21 milestone Nov 27, 2023
@mbien mbien marked this pull request as ready for review November 28, 2023 02:01
@mbien
Copy link
Member Author

mbien commented Nov 28, 2023

@ebarboni can you take a quick look at this, would like to merge since I have something else pending. This includes a change in a sig file too.

@ebarboni
Copy link
Contributor

+1 sorry I was puzzled as I found outdated constants but it's defined https://github.com/apache/maven
DEFAULT_REMOTE_REPO_URL with the apache url.

@mbien mbien merged commit 2b094dd into apache:master Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:all-tests [ci] enable all tests Code cleanup Maven [ci] enable "build tools" tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants