Skip to content

Windows wip#3576

Open
ralight wants to merge 53 commits intomasterfrom
windows-wip
Open

Windows wip#3576
ralight wants to merge 53 commits intomasterfrom
windows-wip

Conversation

@ralight
Copy link
Copy Markdown
Contributor

@ralight ralight commented Apr 27, 2026

Thank you for contributing your time to the Mosquitto project!

Before you go any further, please note that we cannot accept contributions if
you haven't signed the Eclipse Contributor Agreement.
If you aren't able to do that, or just don't want to, please describe your bug
fix/feature change in an issue. For simple bug fixes it is can be just as easy
for us to be told about the problem and then go fix it directly.

Then please check the following list of things we ask for in your pull request:

  • Have you signed the Eclipse Contributor Agreement, using the same email address as you used in your commits?
  • Do each of your commits have a "Signed-off-by" line, with the correct email address? Use "git commit -s" to generate this line for you.
  • If you are contributing a new feature, is your work based off the develop branch?
  • If you are contributing a bugfix, is your work based off the fixes branch?
  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you successfully run make test with your changes locally?

@ralight ralight force-pushed the windows-wip branch 9 times, most recently from da4204c to ae5db58 Compare April 28, 2026 08:02
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 28, 2026

Codecov Report

❌ Patch coverage is 90.00000% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
client/client_shared.c 77.77% 2 Missing ⚠️
Files with missing lines Coverage Δ
apps/mosquitto_passwd/get_password.c 87.80% <ø> (ø)
client/client_props.c 83.96% <100.00%> (ø)
libcommon/file_common.c 64.90% <100.00%> (ø)
src/logging.c 72.72% <ø> (ø)
src/mosquitto.c 51.01% <100.00%> (ø)
src/signals.c 69.64% <ø> (ø)
client/client_shared.c 79.05% <77.77%> (+0.24%) ⬆️

... and 17 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ralight ralight force-pushed the windows-wip branch 16 times, most recently from ca1993a to 2751b7e Compare April 30, 2026 23:19
ralight added 29 commits May 1, 2026 13:28
It doesn't work well with ccache in CI
Sqlite uses contextlib for opening connections
Checks can optionally pass an existing connection so the db only needs
to be opened once even in the full check_db case
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.

1 participant