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

How I can update for formidable to v3. #833

Closed
luuphuc0602 opened this issue Apr 23, 2024 · 11 comments
Closed

How I can update for formidable to v3. #833

luuphuc0602 opened this issue Apr 23, 2024 · 11 comments

Comments

@luuphuc0602
Copy link

No description provided.

@CameronWard301
Copy link

Formidable is added from the "Superagent" dependency, It looks like they are updating that soon:
ladjs/superagent#1799

@AlabasterAxe
Copy link

AlabasterAxe commented Apr 23, 2024

The new version of superagent was just published btw

@melquifrancaminu
Copy link

melquifrancaminu commented Apr 23, 2024

Hi! Eagerly awaiting the new version.

@titanism
Copy link
Collaborator

We're going to do a major bump here too shortly. Working on it now. v9.0.0 of superagent was just released by us.

Consider supporting us at https://forwardemail.net

@titanism
Copy link
Collaborator

Looks like there's a bug after version bumping, haven't got to the bottom of it yet. If anyone wants to help debug please do:

❯ npm test

> [email protected] pretest
> npm run lint --if-present


> [email protected] lint
> eslint lib/**/*.js test/**/*.js index.js


> [email protected] test
> nyc --reporter=html --reporter=text mocha --exit --require should --reporter spec --check-leaks


 Exception during run: TypeError: Cannot read properties of undefined (reading 'push')
    at Agent.<computed> [as set] (/Users/user/Projects/supertest/node_modules/.pnpm/[email protected]/node_modules/superagent/lib/agent-base.js:20:20)
    at Suite.<anonymous> (/Users/user/Projects/supertest/test/supertest.js:952:6)
    at Object.create (/Users/user/Projects/supertest/node_modules/.pnpm/[email protected]/node_modules/mocha/lib/interfaces/common.js:148:19)
    at context.describe.context.context (/Users/user/Projects/supertest/node_modules/.pnpm/[email protected]/node_modules/mocha/lib/interfaces/bdd.js:42:27)
    at Object.<anonymous> (/Users/user/Projects/supertest/test/supertest.js:949:1)
    at Module._compile (node:internal/modules/cjs/loader:1254:14)
    at Module.replacementCompile (/Users/user/Projects/supertest/node_modules/.pnpm/[email protected]/node_modules/append-transform/index.js:60:13)
    at Module._extensions..js (node:internal/modules/cjs/loader:1308:10)
    at Object.<anonymous> (/Users/user/Projects/supertest/node_modules/.pnpm/[email protected]/node_modules/append-transform/index.js:64:4)
    at Module.load (node:internal/modules/cjs/loader:1117:32)
    at Module._load (node:internal/modules/cjs/loader:958:12)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/translators:169:29)
    at ModuleJob.run (node:internal/modules/esm/module_job:194:25)
---------------|---------|----------|---------|---------|-------------------
File           | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s
---------------|---------|----------|---------|---------|-------------------
All files      |   20.58 |      4.8 |    8.69 |   21.11 |
 supertest     |    37.5 |        0 |       0 |    37.5 |
  index.js     |    37.5 |        0 |       0 |    37.5 | 26-54
 supertest/lib |    17.8 |     5.26 |      10 |   18.24 |
  agent.js     |   55.55 |    38.46 |      50 |   54.28 | 34-39,55-56,62-77
  test.js      |    5.45 |        0 |       0 |    5.88 | 26-339
---------------|---------|----------|---------|---------|-------------------

If you want to fork / clone / pull latest commit of this repo, I pushed the version bump and it's broken now

@luan-dang-techlabs
Copy link

i noticed the version wasn't updated properly.

i see this in the npm list tree

@Bruception
Copy link

Bruception commented Apr 23, 2024

i noticed the version wasn't updated properly.

i see this in the npm list tree

They still haven't released the new major version.

@Bruception
Copy link

@titanism I managed to fix this:
Screenshot 2024-04-23 at 7 39 17 PM

@akozlov75
Copy link

This morning npm audit start throwing this issue
image

@hsharif-rh
Copy link

Screenshot 2024-04-24 at 10 28 53 AM

Wondering when we'll see this one resolved...

@titanism
Copy link
Collaborator

v7.0.0 released to npm

https://github.com/ladjs/supertest/releases/tag/v7.0.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

9 participants