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

fix: redirect routing includes protocol & update deps #81

Merged
merged 2 commits into from
Mar 27, 2024

Conversation

SgtPooki
Copy link
Member

@SgtPooki SgtPooki commented Mar 27, 2024

Title

deps: update all deps with npm-check-updates

Description

Ran ncu -u then npm i.

Notes & open questions

also fixed a lint error: /Users/sgtpooki/code/work/protocol.ai/ipfs/helia-http-gateway/src/index.ts:42:7: Irregular whitespace not allowed. [Error/no-irregular-whitespace]

tested with npm run lint && npm run dep-check and will be testing against tiros shortly as well. see #18 (comment)

npm-check-updates output:

╰─ ✔ ❯ ncu -u

   ╭────────────────────────────────────────────────────────────────────────────────────╮
   │                                                                                    │
   │                        Update available 16.14.5 → 16.14.18                         │
   │                      Run npm i -g npm-check-updates to update                      │
   │   https://github.com/raineorshine/npm-check-updates/compare/v16.14.5...v16.14.18   │
   │                                                                                    │
   ╰────────────────────────────────────────────────────────────────────────────────────╯

Upgrading /Users/sgtpooki/code/work/protocol.ai/ipfs/helia-http-gateway/package.json
[====================] 49/49 100%

 @fastify/compress           ^7.0.0  →     ^7.0.1
 @helia/block-brokers        ^2.0.2  →     ^2.0.3
 @helia/interface            ^4.0.1  →     ^4.1.0
 @helia/routers              ^1.0.1  →     ^1.0.2
 @helia/verified-fetch       ^1.1.2  →     ^1.3.2
 @libp2p/webrtc             ^4.0.20  →    ^4.0.22
 @multiformats/multiaddr    ^12.2.0  →    ^12.2.1
 @playwright/test           ^1.40.1  →    ^1.42.1
 @types/node              ^20.11.25  →  ^20.11.30
 dotenv                     ^16.3.1  →    ^16.4.5
 libp2p                      ^1.2.4  →     ^1.3.0
 multiformats               ^13.0.1  →    ^13.1.0
 pino-pretty                ^10.3.1  →    ^11.0.0
 wait-on                     ^7.1.0  →     ^7.2.0

Run npm install to install new versions.

Change checklist

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if necessary (this includes comments as well)
  • I have added tests that prove my fix is effective or that my feature works

Copy link
Member Author

@SgtPooki SgtPooki left a comment

Choose a reason for hiding this comment

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

self review

@@ -39,7 +39,7 @@
*
* ## Run without Docker
*
* ### Build
* ### Build
Copy link
Member Author

Choose a reason for hiding this comment

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

irregular whitespace...?

@SgtPooki
Copy link
Member Author

this is working against tiros locally, but I had to fix redirect routing. see b1119fe (#81)

@SgtPooki SgtPooki changed the title deps: update all deps with npm-check-updates fix: redirect routing & update deps Mar 27, 2024
@SgtPooki SgtPooki changed the title fix: redirect routing & update deps fix: redirect routing includes protocol & update deps Mar 27, 2024
@SgtPooki SgtPooki merged commit 4e332bd into main Mar 27, 2024
6 checks passed
@SgtPooki SgtPooki deleted the deps/update-all branch March 27, 2024 21:58
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