Skip to content

fix(deps): update dependency cookie to v1 #2348

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

Merged
merged 1 commit into from
May 14, 2025
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion docs/package.json
Original file line number Diff line number Diff line change
@@ -38,7 +38,7 @@
"@vercel/analytics": "1.4.1",
"@vercel/remix": "2.15.2",
"clsx": "2.1.1",
"cookie": "0.7.2",
"cookie": "1.0.2",
"isbot": "5.1.19",
"phosphor-react": "1.4.1",
"react": "18.3.1",
10 changes: 5 additions & 5 deletions yarn.lock
Original file line number Diff line number Diff line change
@@ -4691,7 +4691,7 @@ __metadata:
chokidar: 4.0.3
clsx: 2.1.1
concurrently: 9.1.2
cookie: 0.7.2
cookie: 1.0.2
esbuild: 0.24.2
esbuild-register: 3.6.0
gray-matter: 4.0.3
@@ -8287,10 +8287,10 @@ __metadata:
languageName: node
linkType: hard

"cookie@npm:0.7.2":
version: 0.7.2
resolution: "cookie@npm:0.7.2"
checksum: 9bf8555e33530affd571ea37b615ccad9b9a34febbf2c950c86787088eb00a8973690833b0f8ebd6b69b753c62669ea60cec89178c1fb007bf0749abed74f93e
"cookie@npm:1.0.2":
version: 1.0.2
resolution: "cookie@npm:1.0.2"
checksum: 2c5a6214147ffa7135ce41860c781de17e93128689b0d080d3116468274b3593b607bcd462ac210d3a61f081db3d3b09ae106e18d60b1f529580e95cf2db8a55
languageName: node
linkType: hard


Unchanged files with check annotations Beta

}
// The animated prop value of a React element
type AnimatedProp<T> = [T, T] extends [infer T, infer DT]

Check warning on line 31 in targets/zdog/src/animated.ts

GitHub Actions / Style Checks

'T' is defined but never used. Allowed unused vars must match /^_/u
? [DT] extends [never]
? never
: DT extends void
: never
// An animated object of style attributes
type AnimatedStyle<T> = [T, T] extends [infer T, infer DT]

Check warning on line 42 in targets/zdog/src/animated.ts

GitHub Actions / Style Checks

'T' is defined but never used. Allowed unused vars must match /^_/u
? DT extends void
? undefined
: [DT] extends [never]
config: { mass: 20, tension: 150, friction: 50 }
}))
React.useEffect(() => void setInterval(() => api.start((i) => ({ ...random(i), delay: i * 40 })), 3000), [])

Check warning on line 34 in demo/src/sandboxes/springy-boxes/src/App.tsx

GitHub Actions / Style Checks

React Hook React.useEffect has a missing dependency: 'api'. Either include it or remove the dependency array
return (
<>
// Create a local ref if a props function or deps array is ever passed.
const ref = useMemo(
() => (propsFn || arguments.length == 3 ? SpringRef() : void 0),
[]

Check warning on line 85 in packages/core/src/hooks/useSprings.ts

GitHub Actions / Style Checks

React Hook useMemo has a missing dependency: 'propsFn'. Either include it or remove the dependency array
)
interface State {
})
},
}),
[]

Check warning on line 127 in packages/core/src/hooks/useSprings.ts

GitHub Actions / Style Checks

React Hook useMemo has a missing dependency: 'forceUpdate'. Either include it or remove the dependency array
)
const ctrls = useRef([...state.ctrls])
ctrls.current.length = length
declareUpdates(prevLength, length)
}, [length])

Check warning on line 147 in packages/core/src/hooks/useSprings.ts

GitHub Actions / Style Checks

React Hook useMemo has missing dependencies: 'declareUpdates', 'prevLength', and 'ref'. Either include them or remove the dependency array
// Update existing controllers when "deps" are changed.
useMemo(() => {
declareUpdates(0, Math.min(prevLength, length))
// @ts-expect-error – we want to allow passing undefined to useMemo
}, deps)

Check warning on line 153 in packages/core/src/hooks/useSprings.ts

GitHub Actions / Style Checks

React Hook useMemo was passed a dependency list that is not an array literal. This means we can't statically verify whether you've passed the correct dependencies

Check warning on line 153 in packages/core/src/hooks/useSprings.ts

GitHub Actions / Style Checks

React Hook useMemo has missing dependencies: 'declareUpdates', 'length', and 'prevLength'. Either include them or remove the dependency array
/** Fill the `updates` array with declarative updates for the given index range. */
function declareUpdates(startIndex: number, endIndex: number) {
// Return a `SpringRef` if a deps array was passed.
const ref = useMemo(
() => (propsFn || arguments.length == 3 ? SpringRef() : void 0),
[]

Check warning on line 94 in packages/core/src/hooks/useTransition.tsx

GitHub Actions / Style Checks

React Hook useMemo has a missing dependency: 'propsFn'. Either include it or remove the dependency array
)
// Every item has its own transition.
(start: StartFn<T>, stop: StopFn<T>): Promise<any> | void
}
type StartFn<T> = InferTarget<T> extends { start: infer T } ? T : never

Check warning on line 35 in packages/core/src/types/functions.ts

GitHub Actions / Style Checks

'T' is defined but never used. Allowed unused vars must match /^_/u
type StopFn<T> = InferTarget<T> extends { stop: infer T } ? T : never
/**