Skip to content

Merge upstream 2026 05 21#5

Merged
FormelessPuppy41 merged 16 commits into
mainfrom
merge-upstream-2026-05-21
May 21, 2026
Merged

Merge upstream 2026 05 21#5
FormelessPuppy41 merged 16 commits into
mainfrom
merge-upstream-2026-05-21

Conversation

@FormelessPuppy41
Copy link
Copy Markdown
Owner

This PR:

  • Closes #xxxx.
  • Adds and passes relevant tests.
  • Has well-formatted code and documentation.
Details

Notes:

Please read our contributing guidelines first.
In particular:

  • You must add tests when making code changes.
    This keeps the code coverage level up, and helps ensure the changes work as intended.
  • When fixing a bug, you must add a test that would produce the bug in the master branch, and then show that it is fixed with the new code.
  • New code additions must be well formatted. Changes should pass the pre-commit workflow, which you can set up locally using pre-commit.
  • Docstring additions must render correctly, including escapes and LaTeX.
  • Finally, it is essential that all contributions in this PR are license-compatible with PyVRP's MIT license.
    Please check that this PR can be included into PyVRP under the MIT license.

N-Wouda and others added 16 commits March 31, 2026 13:35
…yVRP#1090)

* Add failing test

* Fix evaluation of moves that leave a route empty (without clients)

* Remove Proposal::size()

* Implement numClients() for segments
* Bump configure-pages and deploy-pages actions to versions that use Node 24

* Also bump codecov

(codecov@v5 depends on github-script@v7, which runs with Node 20. github-script@v8 runs with Node 24, and codecov@v6 has that dependency)
* Split off data components to separate header and source files

* Split tests
@FormelessPuppy41 FormelessPuppy41 marked this pull request as ready for review May 21, 2026 10:52
@FormelessPuppy41 FormelessPuppy41 merged commit 334cd88 into main May 21, 2026
5 checks passed
@FormelessPuppy41 FormelessPuppy41 deleted the merge-upstream-2026-05-21 branch May 21, 2026 10:52
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.

2 participants