Skip to content

Main.py: Broken link fix#1330

Merged
henrykironde merged 1 commit intoweecology:mainfrom
tangelll:fix-link-clean
Mar 5, 2026
Merged

Main.py: Broken link fix#1330
henrykironde merged 1 commit intoweecology:mainfrom
tangelll:fix-link-clean

Conversation

@tangelll
Copy link
Copy Markdown
Contributor

Description

Since there is a broken link in main.py i think this should be change. I am not sure about the true link but this can be.

Related Issue(s)

Broken link need to change.

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 28, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.34%. Comparing base (c5897ad) to head (ecf3a1c).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1330      +/-   ##
==========================================
+ Coverage   86.97%   87.34%   +0.36%     
==========================================
  Files          24       24              
  Lines        3009     2978      -31     
==========================================
- Hits         2617     2601      -16     
+ Misses        392      377      -15     
Flag Coverage Δ
unittests 87.34% <ø> (+0.36%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

@vickysharma-prog
Copy link
Copy Markdown
Contributor

vickysharma-prog commented Mar 1, 2026

Link fix verified ✓

A couple of small suggestions:

  1. You may want to squash commits before merging
    (git rebase -i HEAD~5) for a cleaner history.
  2. As a future improvement, we could consider adding
    a CI check for broken links (e.g., markdown-link-check).

@tangelll
Copy link
Copy Markdown
Contributor Author

tangelll commented Mar 1, 2026

@ethanwhite Could you please review and approve this for merging?

@henrykironde
Copy link
Copy Markdown
Contributor

@tangelll, thank you for your contribution. Please rebase your changes into a single commit.
We truly appreciate all contributions; however, please note that reviews may take some time. Patience is an important part of collaborative open-source work. Additionally, if you are preparing for GSoC, please be aware that PRs of this nature are not counted toward contributions considered in the selection process. We expect read code changes and no / or minimum AI usage, and better solution design.

Copy link
Copy Markdown
Contributor

@henrykironde henrykironde left a comment

Choose a reason for hiding this comment

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

@tangelll thanks again for updating this link.

LGTM

@henrykironde henrykironde merged commit 5245dcf into weecology:main Mar 5, 2026
7 checks passed
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