OpenINF / openinf-util-object

Common JavaScript object type-related utilities
https://github.com/OpenINF/openinf-util-object#readme
Other
3 stars 3 forks source link

build(deps-dev): rm eslint-plugin-unicorn entirely #358

Closed DerekNonGeneric closed 1 year ago

DerekNonGeneric commented 1 year ago

This PR was automatically created by Snyk using the credentials of a real user.


Snyk has created this PR to upgrade eslint-plugin-unicorn from 46.0.0 to 47.0.0.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.
*Warning:* This is a major version upgrade, and may be a breaking change. - The recommended version is **2 versions** ahead of your current version. - The recommended version was released **a month ago**, on 2023-05-06.
Release notes
Package name: eslint-plugin-unicorn from eslint-plugin-unicorn GitHub release notes
Commit messages
Package name: eslint-plugin-unicorn
  • 6f05ddc 47.0.0
  • 6ee3687 Fix `expiring-todo-comments` (#2077)
  • 0c3ccb1 `prefer-top-level-await`: Ignore `.cjs` files (#2072)
  • a3bc120 Require Node.js 16, enable `prefer-at`, `prefer-event-target`, and `prefer-string-replace-all` in recommended config (#2073)
  • a9337e8 Extend explanations for some rules (#2070)
  • 2bb1a04 Add `prefer-blob-reading-methods` rule (#2065)
  • d90f337 Use `SourceCode#getScope` (#2068)
  • f144330 Meta tweaks
  • 1eca7d0 Switch to MDN links (#2069)
  • 04af765 Use `SourceCode#{getDeclaredVariables,getAncestors}`, require `eslint@>=8.38.0` (#2066)
  • f5beccb `prefer-spread`: Stop checking `Array.from` call with map function (#2064)
  • 8d6d007 `prefer-dom-node-text-content`: Check optional chaining (#2062)
  • 443999b `prefer-dom-node-{append,remove}`: Check optional chaining (#2061)
  • 0f4f492 Update dependencies (#2060)
  • 124bfa7 `prefer-number-properties`: Fix false positives (#2050)
  • e1c50ac Fix broken link
Compare

**Note:** *You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.* For more information: 🧐 [View latest project report](https://app.snyk.io/org/openinf/project/f770a775-4ad3-480b-bb22-6c4c051fd318?utm_source=github&utm_medium=referral&page=upgrade-pr) 🛠 [Adjust upgrade PR settings](https://app.snyk.io/org/openinf/project/f770a775-4ad3-480b-bb22-6c4c051fd318/settings/integration?utm_source=github&utm_medium=referral&page=upgrade-pr) 🔕 [Ignore this dependency or unsubscribe from future upgrade PRs](https://app.snyk.io/org/openinf/project/f770a775-4ad3-480b-bb22-6c4c051fd318/settings/integration?pkg=eslint-plugin-unicorn&utm_source=github&utm_medium=referral&page=upgrade-pr#auto-dep-upgrades)
socket-security[bot] commented 1 year ago

New and updated dependency changes detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives^1 Size Publisher
eslint-plugin-unicorn ⬆️ 46.0.0...47.0.0 None +1/-1 648 kB sindresorhus
DerekNonGeneric commented 1 year ago

Warning: This is a major version upgrade, and may be a breaking change.

The recommended version is 2 versions ahead of your current version. The recommended version was released a month ago, on 2023-05-06.

@lirantal using my credentials for this is abusive (PRs are permanent); please look into it.

DerekNonGeneric commented 1 year ago

So why did we add this dependency in the first place? Simply for helping to migrate towards prevent-abbreviations, and probably some other things, but mainly that.

Socket score not looking good tho.

DerekNonGeneric commented 1 year ago

[…] and probably some other things, but mainly that.

For posterity's take, all of this config is still located here. This would all be getting folded into OpenINF's Style Guide Standards tho.

lirantal commented 1 year ago

Warning: This is a major version upgrade, and may be a breaking change.

The recommended version is 2 versions ahead of your current version. The recommended version was released a month ago, on 2023-05-06.

@lirantal using my credentials for this is abusive (PRs are permanent); please look into it.

@DerekNonGeneric I can only assume that you have at one point connected to Snyk with your GitHub account and added this repository to monitor which it then automated PR security fixes and package updates. Can you login to Snyk and confirm? there you can also choose to remove this repository from receiving such updates

DerekNonGeneric commented 1 year ago

Hi Liran, sorry about the delayed response. Can you confirm the below is how it should look?

image

Refs: https://support.snyk.io/hc/en-us/articles/360000911157-Is-there-a-way-to-stop-Snyk-from-creating-GitHub-pull-requests-with-changes-like-dependency-updates-and-patches-