Skip to content

876/vInt - #1273

Merged
stevepiercy merged 4 commits into
collective:mainfrom
ijsbol:876/vInt
Mar 12, 2026
Merged

876/vInt#1273
stevepiercy merged 4 commits into
collective:mainfrom
ijsbol:876/vInt

Conversation

@ijsbol

@ijsbol ijsbol commented Mar 12, 2026

Copy link
Copy Markdown
Contributor

Description

Follows on from #1265 regarding #876

Checklist

  • I've added a change log entry to CHANGES.rst.
  • I've added or updated tests if applicable.
  • I've run and ensured all tests pass locally by following Run tests.
  • I've added or edited documentation, both as docstrings to be rendered in the API documentation and narrative documentation, as necessary.

@ijsbol ijsbol mentioned this pull request Mar 12, 2026
4 tasks
@ijsbol

ijsbol commented Mar 12, 2026

Copy link
Copy Markdown
Contributor Author

There are still 20 items left on #876 - should I continue to create individual PRs for each addition, or would you prefer one PR with the additional features? cc @niccokunzmann

@stevepiercy

Copy link
Copy Markdown
Member

@ijsbol one at a time is easier to review and will get merged more quickly.

@stevepiercy stevepiercy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you!

@stevepiercy
stevepiercy merged commit 09ec119 into collective:main Mar 12, 2026
16 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.

2 participants