Convert smiTest1 to catch by ricrogz · Pull Request #9004 · rdkit/rdkit (original) (raw)

Skip to content

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@ricrogz

Not much more to say: converts the test to catch2 format, and renames the source file it so it is easier to identify.

@ricrogz

greglandrum

Choose a reason for hiding this comment

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

LGTM

@ricrogz ricrogz deleted the convert_smiTest1_to_catch branch

December 15, 2025 10:48

greglandrum pushed a commit that referenced this pull request

Dec 30, 2025

@ricrogz @greglandrum

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

Labels

Cleanup

Code cleanup and refactoring

2 participants

@ricrogz @greglandrum