Skip to content

Comments

chore: update dependencies#180

Merged
robertsLando merged 5 commits intomoscajs:masterfrom
joaovitorteixeira:update-dependencies
Oct 16, 2025
Merged

chore: update dependencies#180
robertsLando merged 5 commits intomoscajs:masterfrom
joaovitorteixeira:update-dependencies

Conversation

@joaovitorteixeira
Copy link
Contributor

@joaovitorteixeira joaovitorteixeira commented Oct 13, 2025

Changes

  • Updated dependencies that were safe to bump:
    • mqemitter-redis
    • pino
    • pino-pretty
    • release-it
  • Updated action node version to 22 and 24
  • Updated Docker image to node 22

This PR closes #179

@robertsLando robertsLando changed the title Update dependencies chore: update dependencies Oct 13, 2025
@robertsLando robertsLando marked this pull request as ready for review October 13, 2025 09:34
@joaovitorteixeira
Copy link
Contributor Author

joaovitorteixeira commented Oct 13, 2025

hey @robertsLando, I have fixed all the tests locally:

File % Stmts % Branch % Funcs % Lines Uncovered Line #s
All files 96.91 84.68 95.83 96.91
aedes-cli/lib 96.79 84.54 95.83 96.79
authorizer.js 99.42 89.28 90.9 99.42 106
cli.js 96 86.88 100 96 215-216,239,298,300-301,329,333,338,343,348,353,358,363,368,373,378,383,409-410
hasher.js 95.12 77.77 100 95.12 19,35
persistence.js 95.74 60 100 95.74 22,35

🌈 TEST COMPLETE 🌈

Asserts: 31 pass 0 fail 31 of 31 complete
Suites: 3 pass 0 fail 3 of 3 complete

@joaovitorteixeira
Copy link
Contributor Author

@robertsLando, I had problems with my private registry that is the reason it was failling, but now tests are working in actions too:
https://github.com/joaovitorteixeira/aedes-cli/actions/runs/18485598627/job/52668470599

@robertsLando
Copy link
Member

@joaovitorteixeira seems that IO have some isssues with dockerhub, I will look at this as soon as possible

@joaovitorteixeira
Copy link
Contributor Author

joaovitorteixeira commented Oct 15, 2025

@robertsLando okay! let me know if there is anything else I can do to help you 😄

@robertsLando robertsLando merged commit 4f2945e into moscajs:master Oct 16, 2025
5 of 7 checks passed
@joaovitorteixeira joaovitorteixeira deleted the update-dependencies branch October 16, 2025 22:27
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.

[bug] update mqemitter-redis to version v7.2.0

2 participants