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.

Sign up for GitHub

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

Jump to bottom

Fix URL in README #142

Merged
merged 1 commit into from
Jan 8, 2025
Merged

Fix URL in README #142

merged 1 commit into from
Jan 8, 2025

Conversation

ykkd
Copy link
Contributor

@ykkd ykkd commented Jan 8, 2025

This pull request fixes a broken URL in the README.md file.

@ykkd ykkd marked this pull request as draft January 8, 2025 18:09
@ykkd ykkd marked this pull request as ready for review January 8, 2025 18:09
@dfed
Copy link
Owner

dfed commented Jan 8, 2025

Thank you! Now I need to figure out why the Check Markdown links CI job didn't catch this.

@dfed dfed enabled auto-merge (squash) January 8, 2025 18:39
@dfed dfed mentioned this pull request Jan 8, 2025
Merged
@dfed
Copy link
Owner

dfed commented Jan 8, 2025

Looks like I haven't set up code coverage to work from forks. I'll fix that, but it's certainly not blocking this PR. Thank you for the contribution!

@dfed dfed disabled auto-merge January 8, 2025 18:50
@dfed dfed merged commit 4225ff9 into dfed:main Jan 8, 2025
16 of 18 checks passed
@dfed
Copy link
Owner

dfed commented Jan 8, 2025

Markdown linter fix is here: #143
Codecov fix is here: #144

Thank you again for your contribution! We should be much better shape going forward now.

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

@dfed dfed dfed approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ykkd @dfed