Skip to content

Conversation

@RayStick
Copy link
Member

@RayStick RayStick commented Sep 8, 2025

No description provided.

@github-actions github-actions bot added the Documentation This issue or PR is about the documentation label Sep 8, 2025
@RayStick RayStick requested a review from smoia September 8, 2025 13:13
@github-actions

This comment was marked as outdated.

@RayStick RayStick requested a review from m-miedema September 8, 2025 13:13
@github-actions

This comment was marked as outdated.

@RayStick RayStick requested a review from smoia September 8, 2025 13:18
@github-actions
Copy link

github-actions bot commented Sep 8, 2025

Hello @smoia!

You've been assigned to this PR, which means that you've been nominated Main Reviewer! Lucky you!

As a Main Reviewer, you are slightly more responsible for the quality of this PR than your fellow Reviewers. If you have any doubt, check this section on reviewing and this section on being the Main Reviewer of the documents.

Before merging this PR, please check that:

  • The PR passes all the CircleCI/Azure tests.
  • The content respects the Style Guide.
  • If the PR contains documentation, your local build is right.
  • The PR and the reviews are harmonious with the rest of the repository, especially that they don't introduce repetitions
  • The title of the PR is clear enough to describe a release content.
  • The labels are correct - and they will (or won't) trigger the correct auto release.
  • The PR received the amount of approvals necessaries to be merged.

If the PR contains code changes:

  • Content and changes are adequately documented in the docstrings.
  • User documentation is being updated accordingly - or a related issue has been opened.
  • The adequate tests have been added/updated - or a related issue has been opened if coverage doesn't drop below 90%.

After you merged this PR, please check that:

  • The Author(s) and Reviewers contributions have been updated in the README.
  • The updates (in code, tests and documentation) have appened correctly.
  • If they had to, the tag was created, the release was cut, and the pypi version got updated.

Thank you!

@RayStick RayStick requested a review from m-miedema September 8, 2025 13:18
@github-actions
Copy link

github-actions bot commented Sep 8, 2025

Hello @m-miedema!

You've been assigned to this PR, which means that you've been nominated as reviewer! You are a supporting reviewer, not the main reviewer - the main reviewer has more responsibilities than you (lucky them) but your review is still important.

If you have any doubt, check this section on reviewing

Checklist

  • one
  • two
  • three

Thank you!

@RayStick RayStick requested a review from me-pic September 8, 2025 13:19
@github-actions
Copy link

github-actions bot commented Sep 8, 2025

Hello @me-pic!

You've been assigned to this PR, which means that you've been nominated as reviewer! You are a supporting reviewer, not the main reviewer - the main reviewer has more responsibilities than you (lucky them) but your review is still important.

If you have any doubt, check this section on reviewing

Checklist

  • one
  • two
  • three

Thank you!

@RayStick RayStick merged commit 1095d90 into master Sep 8, 2025
4 checks passed
@RayStick RayStick deleted the testing-PR-checklist branch September 8, 2025 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation This issue or PR is about the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants