Skip to content

test: Check for warnings and prepare to enable fixed tests#116

Merged
duhow merged 4 commits into
masterfrom
enable-tests
Nov 4, 2023
Merged

test: Check for warnings and prepare to enable fixed tests#116
duhow merged 4 commits into
masterfrom
enable-tests

Conversation

@duhow

@duhow duhow commented Nov 4, 2023

Copy link
Copy Markdown
Collaborator

Issue #90 is fixed (#111 is unrelated).

@Fubukimaru Fubukimaru self-assigned this Nov 4, 2023
@Fubukimaru Fubukimaru self-requested a review November 4, 2023 11:33
@duhow duhow changed the title Enable fixed test test: Check for warnings and prepare to enable fixed tests Nov 4, 2023

@Fubukimaru Fubukimaru left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Excelentes cambios en los test.
They fulfill the new bug check :)

@duhow duhow merged commit 8f33dd2 into master Nov 4, 2023
@duhow duhow deleted the enable-tests branch November 4, 2023 11:44
Fubukimaru added a commit that referenced this pull request Nov 4, 2023
* chores: Improved test for issue #90
* Add check for warnings

---------

Co-authored-by: Alberto Gutiérrez Torre <alberto.gutierrez@bsc.es>
duhow added a commit that referenced this pull request Nov 4, 2023
* chores: Improved test for issue #90
* Add check for warnings

---------

Co-authored-by: Alberto Gutiérrez Torre <alberto.gutierrez@bsc.es>
duhow added a commit to duhow/asMSX that referenced this pull request Nov 5, 2023
…u#116)

* chores: Improved test for issue Fubukimaru#90
* Add check for warnings

---------

Co-authored-by: Alberto Gutiérrez Torre <alberto.gutierrez@bsc.es>
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.

2 participants