Skip to content

MOLE.jl: remove unneeded dependency#380

Merged
valeriabarra merged 1 commit into
mainfrom
valeria/fix-julia-tests-dep
Jun 3, 2026
Merged

MOLE.jl: remove unneeded dependency#380
valeriabarra merged 1 commit into
mainfrom
valeria/fix-julia-tests-dep

Conversation

@valeriabarra
Copy link
Copy Markdown
Collaborator

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Example
  • Documentation

Description

Description of the bug was provided in the Issue #379

Related Issues & Documents

QA Instructions, Screenshots, Recordings

The correct behavior with this fix is:

Testing Running tests...
Test Summary:          | Pass  Total  Time
Testing MOLE operators |   76     76  6.9s
Test Summary:               | Pass  Total  Time
Testing Boundary Conditions |   14     14  5.2s
     Testing MOLE tests passed 

Reason:

Added/updated tests?

_We encourage you to test all code included with MOLE, including examples.

  • Yes
  • No, and this is why: please replace this line with details on why tests
    have not been included
  • I need help with writing tests

Read Contributing Guide and Code of Conduct

Copy link
Copy Markdown
Collaborator

@mathgeek31415 mathgeek31415 left a comment

Choose a reason for hiding this comment

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

I was able to re-produce the error you identified; and, the update you made looks to have fixed this. Approving this pull request!

@valeriabarra valeriabarra merged commit 701161a into main Jun 3, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MOLE.jl test suite dependency issue

2 participants