feat: project setup #10
Conversation
.github/workflows/main.yml
Outdated
|
|
||
| strategy: | ||
| matrix: | ||
| node-version: [22.x] |
There was a problem hiding this comment.
I suggest to be more permissive with version, following what rhea supports
There was a problem hiding this comment.
alright, i will put the same versions as in rhea workflow https://github.com/amqp/rhea/blob/main/.github/workflows/node.js.yml#L18
There was a problem hiding this comment.
are you sure this file is executed by vite?
There was a problem hiding this comment.
added the file to the list of setup files and set sequence setup to list, so we can decide the order to run the files
There was a problem hiding this comment.
are you sure this file is executed by vite?
There was a problem hiding this comment.
added the file to the list of setup files and set sequence setup to list, so we can decide the order to run the files
test/e2e/management.test.ts
Outdated
| management.close() | ||
| }) | ||
|
|
||
| test("create a queue through the management", async () => { |
There was a problem hiding this comment.
Set as skip so we can merge without broken test
test/tsconfig.json
Outdated
There was a problem hiding this comment.
Are you sure it's necessary this file? with the tsconfi.json that include include": ["src/**/*", "test/**/*", "vitest.config.mts"], I don't think it's necessary
Add basic project setup using node and esm, replacing mocha with vitest for esm compatibility