Skip to content

Conversation

nagu165
Copy link

@nagu165 nagu165 commented Nov 15, 2023

I modified the ESLint package and also made some minor adjustments to index.js file.
Thank you.

Copy link

New dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
eslint 8.53.0 eval, filesystem +72 10.4 MB eslintbot
Copy link
Member

@ljharb ljharb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why? I prefer the eslintrc format; nothing needs updating.

"aud": "^2.0.3",
"auto-changelog": "^2.4.0",
"eslint": "=8.8.0",
"eslint": "^8.8.0",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this makes the peer dep invalid; 8.8.0 is the last version that properly parses ES3 code with the strict pragma.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes I understand the issue now .
Thankyou for the response I will try to improve upon my shortcomings.

This comment was marked as spam.

'use strict';

var crypto = require('crypto');
const crypto = require('crypto');
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is a breaking change for pre-const engines.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah an oversight on my part.
Thanks for the response.

@ljharb ljharb marked this pull request as draft November 15, 2023 22:00
@nagu165 nagu165 closed this by deleting the head repository Jan 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants