Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Fix the condition #386

Merged
merged 7 commits into from
Jan 10, 2024
Merged

Fix the condition #386

merged 7 commits into from
Jan 10, 2024

Conversation

artjimlop
Copy link
Collaborator

Captura de pantalla 2024-01-10 a las 15 08 56

I didn't notice the negation was missing. Amazing. Kill me please.

@artjimlop artjimlop changed the title Fix the shit Fix the condition Jan 10, 2024
@artjimlop artjimlop marked this pull request as ready for review January 10, 2024 14:12
@artjimlop artjimlop requested a review from ferranpons as a code owner January 10, 2024 14:12
ferranpons
ferranpons approved these changes Jan 10, 2024
View reviewed changes
Copy link
Member

@ferranpons ferranpons left a comment

Choose a reason for hiding this comment

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

😅

@ferranpons ferranpons merged commit f77114e into master Jan 10, 2024
1 check passed
@delete-merged-branch delete-merged-branch bot deleted the SCMI-112253-no-coordinates-in-bundle-show-results branch January 10, 2024 16:08
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@ferranpons ferranpons ferranpons approved these changes

@jlquintana jlquintana Awaiting requested review from jlquintana

@marcserrascmspain marcserrascmspain Awaiting requested review from marcserrascmspain

Assignees

@ferranpons ferranpons

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@artjimlop @ferranpons