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

Many changes #166

Merged
merged 16 commits into from Oct 27, 2022
Merged
Show file tree
Hide file tree
Changes from 11 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 2 additions & 0 deletions .eslintrc.js
Expand Up @@ -36,5 +36,7 @@ module.exports = {
'@typescript-eslint/explicit-module-boundary-types': 0,
'@typescript-eslint/no-explicit-any': 0,
'no-console': 0,
'no-shadow': 0,
'@typescript-eslint/no-shadow': 1,
},
};
6 changes: 3 additions & 3 deletions .github/workflows/lint.yml
Expand Up @@ -10,11 +10,11 @@ jobs:
lint:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v3
- name: Setup Node.js
uses: actions/setup-node@v1.4.2
uses: actions/setup-node@v3.4.1
with:
node-version: 13
node-version: 16
- name: Install dependencies
run: npm i
- name: Run lint
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/test-coverage.yml
Expand Up @@ -10,11 +10,11 @@ jobs:
test-coverage:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v3
- name: Setup Node.js
uses: actions/setup-node@v1.4.2
uses: actions/setup-node@v3.4.1
with:
node-version: 13
node-version: 16
- name: Setup Redis
uses: zhulik/redis-action@1.1.0
with:
Expand Down
10 changes: 5 additions & 5 deletions .github/workflows/tests.yml
Expand Up @@ -8,17 +8,17 @@ on:

jobs:
tests:
name: "tests: Node ${{ matrix.node_version }} Redis ${{ matrix.redis_version }}"
name: 'tests: Node ${{ matrix.node_version }} Redis ${{ matrix.redis_version }}'
runs-on: ubuntu-latest
strategy:
fail-fast: false
matrix:
node_version: ["10", "12", "13"]
redis_version: ["4", "5"]
node_version: ['10', '12', '13']
redis_version: ['4', '5']
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v3
- name: Setup Node.js ${{ matrix.node_version }}
uses: actions/setup-node@v1.4.2
uses: actions/setup-node@v3.4.1
with:
node-version: ${{ matrix.node_version }}
- name: Setup Redis ${{ matrix.redis_version }}
Expand Down
2 changes: 2 additions & 0 deletions .gitignore
Expand Up @@ -65,3 +65,5 @@ dist
sandpit

.DS_Store

.vscode
6 changes: 3 additions & 3 deletions benchmarks/index.ts
Expand Up @@ -2,15 +2,15 @@
/* eslint-disable no-plusplus */
/* eslint-disable no-await-in-loop */
/* eslint-disable no-restricted-syntax */
import { createManager, createWorker } from 'conveyor-mq';
import BeeQueue from 'bee-queue';
import BullQueue from 'bull';
import { createManager, createWorker } from 'conveyor-mq';

const redisConfig = { host: 'localhost', port: 6379 };

const countdown = (n = 1) => {
let next: () => boolean;
const done = new Promise((resolve) => {
const done = new Promise<void>((resolve) => {
next = () => {
n--;
if (n < 0) return false;
Expand Down Expand Up @@ -73,7 +73,7 @@ const benchmarks = {
handlerTimeout = 0,
}) => {
const queue = new BeeQueue('bee-queue');
const ready = new Promise((resolve) => {
const ready = new Promise<void>((resolve) => {
queue.on('ready', () => {
resolve();
});
Expand Down