Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 4 commits into from
Nov 4, 2023

Conversation

duhow
Copy link
Collaborator

@duhow duhow commented Nov 4, 2023

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
Copy link
Owner

@Fubukimaru Fubukimaru left a comment

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
9 checks passed
@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 <[email protected]>
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 <[email protected]>
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 <[email protected]>
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.

None yet

2 participants