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: correct github commit after link in commit list #2000

Merged
merged 1 commit into from
Sep 8, 2024

Conversation

Spice-King
Copy link
Contributor

Description

Fixes an incorrect GitHub URL when the Moonraker update_manager extension name does not match the GitHub repo name.

Given the following update manager extension config (which is the one LED Effects for Klipper uses)

[update_manager led_effect]
type: git_repo
path: ~/klipper-led_effect
origin: https://github.com/julianschill/klipper-led_effect.git
is_system_service: False

The change corrects the "Link to GitHub" buttton's address from
https://github.com/julianschill/led_effect/commits/master/?after=bde44b49246a0fe7b50d41e415d71e8f966d9542+0
to
https://github.com/julianschill/klipper-led_effect/commits/master/?after=bde44b49246a0fe7b50d41e415d71e8f966d9542+0

Related Tickets & Documents

none

Mobile & Desktop Screenshots/Recordings

none

[optional] Are there any post-deployment tasks we need to perform?

none

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Sep 8, 2024
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Sep 8, 2024
@meteyou meteyou merged commit 55ae60c into mainsail-crew:develop Sep 8, 2024
12 checks passed
@meteyou
Copy link
Member

meteyou commented Sep 8, 2024

thx for fixing this bug!

@Spice-King Spice-King deleted the fix-github-link branch September 8, 2024 16:47
@Spice-King
Copy link
Contributor Author

No problem!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants