Skip to content

Conversation

@xiaoliz0
Copy link
Contributor

New version of IPDXXX_Remisse_draft content from OUS. (issue #86)

@xiaoliz0 xiaoliz0 linked an issue Oct 30, 2025 that may be closed by this pull request
Copy link
Collaborator

@marrip marrip left a comment

Choose a reason for hiding this comment

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

looks good 👍

@marrip
Copy link
Collaborator

marrip commented Nov 10, 2025

I see the unit tests fail - I will take a look. It is probably because you didn't change the strings in the expected section to match the changes you made.

Updated expected output strings in test cases to use capital letters.
@xiaoliz0
Copy link
Contributor Author

I see the unit tests fail - I will take a look. It is probably because you didn't change the strings in the expected section to match the changes you made.

Thanks for your checking and fixing!

@xiaoliz0 xiaoliz0 merged commit 855b64f into main Nov 13, 2025
1 check passed
@xiaoliz0 xiaoliz0 deleted the develop_issue86 branch December 11, 2025 09:42
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.

New version of IPDXXX_Remisse_draft content from OUS

3 participants