Skip to content

Conversation

@alexandre-abrioux
Copy link
Contributor

@alexandre-abrioux alexandre-abrioux commented May 9, 2023

Description of the changes

This PR:

@coveralls
Copy link

coveralls commented May 9, 2023

Coverage Status

Coverage: 87.342%. Remained the same when pulling cf86847 on fix-integration-tests into 35a8e0b on master.

@alexandre-abrioux
Copy link
Contributor Author

@MantisClone @KolevDarko asking for review again, I updated the tests that were broken

Copy link
Member

@MantisClone MantisClone left a comment

Choose a reason for hiding this comment

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

Approved, but just a head's up, one of the utils tests timed out.

@alexandre-abrioux alexandre-abrioux merged commit af5c902 into master May 12, 2023
@alexandre-abrioux alexandre-abrioux deleted the fix-integration-tests branch May 12, 2023 11:50
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.

5 participants