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.

Sign up for GitHub

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

Jump to bottom

[mosquitto]: Fix minor CI issue #753

Merged
merged 1 commit into from
Feb 3, 2025
Merged

[mosquitto]: Fix minor CI issue #753

merged 1 commit into from
Feb 3, 2025

Conversation

david-cermak
Copy link
Collaborator

@david-cermak david-cermak commented Jan 31, 2025 •
edited
Loading

IDF-test was executed even if no mosq label added.

@david-cermak david-cermak self-assigned this Jan 31, 2025
@david-cermak david-cermak requested a review from glmfe January 31, 2025 10:26
@david-cermak
Copy link
Collaborator Author

@glmfe PTAL at the minor fix -- noticed that the mosq-build test was executed in your WSL PR during the review yesterday.

@glmfe
Copy link
Collaborator

glmfe commented Feb 3, 2025

@glmfe PTAL at the minor fix -- noticed that the mosq-build test was executed in your WSL PR during the review yesterday.

Noticed that too, thanks!

@david-cermak david-cermak merged commit 1c6580e into espressif:master Feb 3, 2025
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@glmfe glmfe glmfe approved these changes

Assignees

@david-cermak david-cermak

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@david-cermak @glmfe