Skip to content

Commit

Permalink
remove node 12 support yml
Browse files Browse the repository at this point in the history
bump eslint ecmaVersion to 20

lint
  • Loading branch information
Allan Murara committed Dec 24, 2021
1 parent c155ee1 commit 2e71e77
Show file tree
Hide file tree
Showing 8 changed files with 10 additions and 10 deletions.
2 changes: 1 addition & 1 deletion .eslintrc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ parserOptions:
#ecmaVersion: 2015 (6)
#ecmaVersion: 2016 (7) (node 6)
#ecmaVersion: 2017 (8) (node 8)
ecmaVersion: 2017
ecmaVersion: 2020
#ecmaVersion: 2018 (9) (node 10)
#ecmaVersion: 2019 (10) (node 12)
#ecmaVersion: 2020 (11) (node 14)
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/ci-test-win.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ jobs:
strategy:
matrix:
os: [ windows-latest ]
node-version: [12.x, 14.x]
node-version: [14.x]
fail-fast: false

steps:
Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/ci-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,8 @@ jobs:
# node 6, maint. ended 2018-04
# node 8, maint. ended 2019-12
# node 10, maint. ends 2021-04
node-version: [12.x, 14.x]
# node 12, maint. ends 2022-04
node-version: [14.x]
fail-fast: false

steps:
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/coveralls.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,10 @@ jobs:
with:
fetch-depth: 1

- name: Use Node.js 12
- name: Use Node.js 14
uses: actions/setup-node@master
with:
node-version: 12.x
node-version: 14.x

- name: install, run
run: |
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ jobs:

strategy:
matrix:
node-version: [ 12.x ]
node-version: [ 14.x ]

steps:
- uses: actions/checkout@v1
Expand Down
1 change: 0 additions & 1 deletion plugins/queue/qmail-queue.js
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,6 @@ exports.hook_queue = function (next, connection) {
qmail_queue.stdin.on('close', () => {
if (connection?.transaction == null) {
plugin.logerror("Transaction went away while delivering mail to qmail-queue");

try {
qmail_queue.stdout.end();
}
Expand Down
4 changes: 2 additions & 2 deletions plugins/spf.js
Original file line number Diff line number Diff line change
Expand Up @@ -308,8 +308,8 @@ exports.return_results = function (next, connection, spf, scope, result, sender)
exports.save_to_header = (connection, spf, result, mfrom, host, id, ip) => {
// Add a trace header
if (connection?.transaction == null) {
connection.logwarn(plugin, "save_to_header could not find transaction, returning undefined");
return;
connection.logwarn("save_to_header could not find transaction, returning undefined");
return;
}
const des = result === spf.SPF_PASS ? 'designates' : 'does not designate';
const identity = `identity=${id}; client-ip=${ip ? ip : connection.remote.ip}`;
Expand Down
2 changes: 1 addition & 1 deletion smtp_client.js
Original file line number Diff line number Diff line change
Expand Up @@ -281,7 +281,7 @@ class SMTPClient extends events.EventEmitter {
if (connection?.transaction == null) {
connection.logwarn(plugin, "transaction went away, releasing smtp_client");
this.release();
return true;
return true;
}
return false;
}
Expand Down

0 comments on commit 2e71e77

Please sign in to comment.