-
Notifications
You must be signed in to change notification settings - Fork 2
chore: Bump dependencies #287
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
Changes from 10 commits
53a9f96
4b22610
ec704dc
970fa28
c6c48ff
5ce6c5d
33e546b
44d40ba
975face
d26d454
da643ab
b607f9a
e63ddfe
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1 @@ | ||
#!/bin/sh | ||
. "$(dirname "$0")/_/husky.sh" | ||
|
||
yarn lint-staged |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1 @@ | ||
20 | ||
22 |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,38 @@ | ||
import eslint from "@eslint/js"; | ||
import tseslint from "typescript-eslint"; | ||
import vitest from "@vitest/eslint-plugin"; | ||
import globals from "globals"; | ||
|
||
// @ts-check | ||
|
||
export default tseslint.config( | ||
{ ignores: ["dist"] }, | ||
eslint.configs.recommended, | ||
tseslint.configs.recommended, | ||
{ | ||
files: ["test/**/*.{ts,js}", "vitest.setup.js", "vitest.globalsetup.js"], | ||
...vitest.configs.recommended, | ||
languageOptions: { | ||
globals: { | ||
...globals.browser, | ||
...globals.node, | ||
...globals.es2025, | ||
...vitest.environments.env.globals, | ||
}, | ||
}, | ||
}, | ||
{ | ||
files: ["source/**/*.{ts,js}"], | ||
languageOptions: { | ||
globals: { ...globals.browser, ...globals.node, ...globals.es2025 }, | ||
}, | ||
}, | ||
{ | ||
files: ["**/*.ts"], | ||
rules: { | ||
"@typescript-eslint/no-explicit-any": "off", | ||
"@typescript-eslint/no-empty-object-type": "off", | ||
"@typescript-eslint/ban-ts-comment": "off", | ||
}, | ||
}, | ||
); |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -306,7 +306,7 @@ export class Session< | |
if (data && data.constructor === Object) { | ||
const out: Data = {}; | ||
for (const key in data) { | ||
if (data.hasOwnProperty(key)) { | ||
if (Object.prototype.hasOwnProperty.call(data, key)) { | ||
jimmycallin marked this conversation as resolved.
Show resolved
Hide resolved
|
||
out[key] = this.encode(data[key]); | ||
} | ||
} | ||
|
@@ -545,7 +545,7 @@ export class Session< | |
const mergedEntity = identityMap[identifier]; | ||
|
||
for (const key in entity) { | ||
if (entity.hasOwnProperty(key)) { | ||
if (Object.prototype.hasOwnProperty.call(entity, key)) { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. eslint - avoid calling hasOwnProperty on the object directly There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Use Object.hasOwn() instead. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. fixed! |
||
mergedEntity[key] = this.decode(entity[key], identityMap, { | ||
decodeDatesAsIso, | ||
ensureSerializableResponse, | ||
|
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
husky deprecated having to include this