Skip to content

Change Maven hostname to maven.pkg.github.com#653

Merged
lucascosti merged 6 commits intomainfrom
maven-pkg
Nov 4, 2020
Merged

Change Maven hostname to maven.pkg.github.com#653
lucascosti merged 6 commits intomainfrom
maven-pkg

Conversation

@jcansdale
Copy link
Copy Markdown
Contributor

@jcansdale jcansdale commented Oct 19, 2020

Why:

Make Maven hostname consistent with other ecosystems (maven.pkg.github.com).

As suggested here https://github.zendesk.com/agent/tickets/851339

What's being changed:

  • Change Maven host to maven.pkg.github.com.
  • Change rubygem.pkg.github.com to rubygems.pkg.github.com.
  • Add missing tonuget.pkg.github.com`.

Check off the following:

@janiceilene
Copy link
Copy Markdown
Contributor

👋 Thanks for opening a PR @jcansdale! Would you mind resolving the conflicts? I'll get this triaged for review ⚡

@janiceilene janiceilene added content This issue or pull request belongs to the Docs Content team ecosystem This issue or pull request should be reviewed by the Docs Ecosystem team packages This issue or pull request should be reviewed by the docs packages team labels Oct 20, 2020
@jcansdale
Copy link
Copy Markdown
Contributor Author

@janiceilene done. 😄 I don't know why that was conflicted!

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 2, 2020

This PR is stale because it has been open 7 days with no activity and will be automatically closed in 3 days. To keep this PR open, update the PR by adding a comment or pushing a commit.

@github-actions github-actions Bot added the stale There is no recent activity on this issue or pull request label Nov 2, 2020
@jcansdale jcansdale removed the stale There is no recent activity on this issue or pull request label Nov 2, 2020
@lucascosti lucascosti self-assigned this Nov 4, 2020
Copy link
Copy Markdown
Contributor

@lucascosti lucascosti left a comment

Choose a reason for hiding this comment

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

Thanks a lot for these fixes, @jcansdale. I'll get them merged in 🚀

Comment on lines -106 to -107
| Java | Apache Maven project management and comprehension tool | `pom.xml` | `mvn` | `maven.HOSTNAME/OWNER/REPOSITORY/IMAGE-NAME`
| Java | Gradle build automation tool for Java | `build.gradle` or `build.gradle.kts` | `gradle` | `maven.HOSTNAME/OWNER/REPOSITORY/IMAGE-NAME`
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

😬 eeep, looks like this probably crept in from the GHES versioning...

@lucascosti lucascosti merged commit 66cb3fc into main Nov 4, 2020
@lucascosti lucascosti deleted the maven-pkg branch November 4, 2020 04:27
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 4, 2020

Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content This issue or pull request belongs to the Docs Content team ecosystem This issue or pull request should be reviewed by the Docs Ecosystem team packages This issue or pull request should be reviewed by the docs packages team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants