Skip to content

Link checker#196

Open
arnavk23 wants to merge 7 commits intoJuliaSmoothOptimizers:mainfrom
arnavk23:main-only-clean
Open

Link checker#196
arnavk23 wants to merge 7 commits intoJuliaSmoothOptimizers:mainfrom
arnavk23:main-only-clean

Conversation

@arnavk23
Copy link
Copy Markdown
Contributor

@arnavk23 arnavk23 commented Apr 6, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 6, 2026 18:50
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR focuses on making documentation and GitHub templates more robust to automated link checking by updating or excluding problematic URLs.

Changes:

  • Update contributing-guide links in README.md and GitHub issue templates to point to the repository source file.
  • Fix a broken GitHub link in the benchmark documentation by using the correct blob/main URL.
  • Adjust link-checker configuration (.lychee.toml) to exclude additional URLs.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
README.md Updates the contributing link to a GitHub-hosted doc source path.
docs/src/91-developer.md Updates the “Utopic Git History” reference URL.
docs/src/2-benchmark.md Fixes the benchmark/figures.jl link to use a valid GitHub blob/main URL.
.lychee.toml Adds new URL exclusions for the link checker.
.github/ISSUE_TEMPLATE/99-general.yml Updates contributing-guide link to point to repo source.
.github/ISSUE_TEMPLATE/30-usage.yml Updates contributing-guide link to point to repo source.
.github/ISSUE_TEMPLATE/20-feature-request.yml Updates contributing-guide link to point to repo source.
.github/ISSUE_TEMPLATE/10-bug-report.yml Updates contributing-guide link to point to repo source.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/src/2-benchmark.md Outdated
Comment thread .lychee.toml Outdated
@tmigot
Copy link
Copy Markdown
Member

tmigot commented Apr 7, 2026

The Link checker stills fails though

arnavk23 and others added 3 commits April 7, 2026 19:12
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@arnavk23
Copy link
Copy Markdown
Contributor Author

arnavk23 commented Apr 7, 2026

@tmigot could you check again. Only one was at fault, so I think it will pass now.

@arnavk23
Copy link
Copy Markdown
Contributor Author

arnavk23 commented Apr 7, 2026

@tmigot Let's try one more time. There shouldn't be a problem (but let's see)

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.52%. Comparing base (f3da629) to head (9fd6fa7).
⚠️ Report is 42 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #196      +/-   ##
==========================================
+ Coverage   82.97%   83.52%   +0.55%     
==========================================
  Files          10       10              
  Lines         599      607       +8     
==========================================
+ Hits          497      507      +10     
+ Misses        102      100       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@arnavk23
Copy link
Copy Markdown
Contributor Author

@tmigot Fixed.

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.

3 participants