Skip to content

Conversation

@gergelyattilakiss
Copy link
Collaborator

Summary

Corrections for review.

Related Issues

Number of Passing Unit Tests

Tests passing: 520/520

Checklist

  • Version number has been bumped in Project.toml

Tests

  • ] test runs without errors
  • Unit tests have been added
  • This is a refactoring, no new tests required

Documentation

  • Documentation has been updated
  • No change to public API, documentation not required

@gergelyattilakiss gergelyattilakiss self-assigned this Mar 6, 2025
@codecov
Copy link

codecov bot commented Mar 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Files with missing lines Coverage Δ
src/functions.jl 87.50% <100.00%> (ø)
src/side_effects.jl 100.00% <ø> (ø)
🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gergelyattilakiss
Copy link
Collaborator Author

@korenmiklos Please check the PR and merge it to main so we can close the review issues and resubmit the paper.

Copy link
Member

@korenmiklos korenmiklos left a comment

Choose a reason for hiding this comment

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

Please bump version number.

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