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

Fix broken docker hub url #1951

Merged
merged 3 commits into from
Nov 29, 2024
Merged

Fix broken docker hub url #1951

merged 3 commits into from
Nov 29, 2024

Conversation

arshadkazmi42
Copy link
Contributor

Hey, I just made a Pull Request!

There was a docker hub url mentioned in the README file. The docker hub url format was wrong and was redirecting to a 404 page.

This PR, fixes the URL

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)
  • All your commits have a Signed-off-by line in the message. (more info)

Signed-off-by: Arshad Kazmi <arshadkazmi42@gmail.com>
@arshadkazmi42 arshadkazmi42 requested a review from a team as a code owner November 16, 2024 19:17
@backstage-goalie
Copy link
Contributor

backstage-goalie bot commented Nov 16, 2024

Changed Packages

Package Name Package Path Changeset Bump Current Version
@backstage-community/plugin-gitops-profiles workspaces/gitops-profiles/plugins/gitops-profiles patch v0.3.54

@christoph-jerolimov christoph-jerolimov enabled auto-merge (squash) November 16, 2024 23:07
Copy link
Contributor

@awanlin awanlin left a comment

Choose a reason for hiding this comment

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

Thanks for the correction @arshadkazmi42, as this README gets published to NPM we do need a changeset for this. Details on how to do that are here: https://github.com/backstage/community-plugins/blob/main/CONTRIBUTING.md#creating-changesets

auto-merge was automatically disabled November 29, 2024 19:40

Head branch was pushed to by a user without write access

@backstage-goalie
Copy link
Contributor

Thanks for the contribution!
All commits need to be DCO signed before they are reviewed. Please refer to the the DCO section in CONTRIBUTING.md or the DCO status for more info.

Signed-off-by: Arshad Kazmi <arshadkazmi42@gmail.com>
@arshadkazmi42
Copy link
Contributor Author

@awanlin I added the changset file

@awanlin awanlin enabled auto-merge (squash) November 29, 2024 19:45
@awanlin
Copy link
Contributor

awanlin commented Nov 29, 2024

Thanks @arshadkazmi42, I've approved and once the build passes this will get merged in 🚀

@awanlin awanlin merged commit 6cd4e02 into backstage:main Nov 29, 2024
11 checks passed
@arshadkazmi42 arshadkazmi42 deleted the patch-1 branch November 29, 2024 19:49
CryptoRodeo pushed a commit to CryptoRodeo/community-plugins that referenced this pull request Dec 10, 2024
* Fix broken docker hub url

Signed-off-by: Arshad Kazmi <arshadkazmi42@gmail.com>

* Added changeset

Signed-off-by: Arshad Kazmi <arshadkazmi42@gmail.com>

---------

Signed-off-by: Arshad Kazmi <arshadkazmi42@gmail.com>
Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants