Express: var statuses = require('statuses') on line 24 missing semicolon

Created on 29 May 2020  路  2Comments  路  Source: expressjs/express

on the response.js semicolon is missing.
https://github.com/expressjs/express/blob/master/lib/response.js

var onFinished = require('on-finished');
var path = require('path');
var statuses = require('statuses')
var merge = require('utils-merge');

Can we keep this consistent? so is there any problem adding the semicolon or is it purposefully not added semicolon?

question

Most helpful comment

Hi @phpjsnerd . This project is trying to convert from no format standard to StandardJS. The StandardJS format does not use semi colons, and in order not to break all outstanding PRs, we have been just changing the lines changed in PRs to StandardJS style. That line without a semicolon was changed/added after we begain migration to StandardJS style.

All 2 comments

Hi @phpjsnerd . This project is trying to convert from no format standard to StandardJS. The StandardJS format does not use semi colons, and in order not to break all outstanding PRs, we have been just changing the lines changed in PRs to StandardJS style. That line without a semicolon was changed/added after we begain migration to StandardJS style.

Thank you for the quick response. You're doing great job!

Was this page helpful?
0 / 5 - 0 ratings

Related issues

despairblue picture despairblue  路  3Comments

cuni0716 picture cuni0716  路  3Comments

afanasy picture afanasy  路  3Comments

ZeddYu picture ZeddYu  路  3Comments

Sunriselegacy picture Sunriselegacy  路  3Comments