Skip to content
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

chore(deps): update dependency tsup to v8 #356

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
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
1,441 changes: 1,301 additions & 140 deletions package-lock.json

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion packages/nidalee/package.json
Original file line number Diff line number Diff line change
@@ -75,7 +75,7 @@
"postcss": "^8.4.31",
"postcss-preset-env": "^9.1.4",
"tsconfig": "*",
"tsup": "^7.2.0",
"tsup": "^8.3.5",
"typescript": "^5.2.2"
}
}

Unchanged files with check annotations Beta

import {render, RenderOptions} from '@testing-library/react';
import userEvent from '@testing-library/user-event';

Check warning on line 2 in packages/nidalee/scripts/testUtils.ts

GitHub Actions / build (18.x)

Using exported name 'userEvent' as identifier for default export
const customRender = (ui: React.ReactElement, options?: RenderOptions) => ({
user: userEvent.setup(),
});
// re-export everything
export * from '@testing-library/react';

Check warning on line 10 in packages/nidalee/scripts/testUtils.ts

GitHub Actions / build (18.x)

Multiple exports of name 'render'
// override render method
export {customRender as render};

Check warning on line 13 in packages/nidalee/scripts/testUtils.ts

GitHub Actions / build (18.x)

Multiple exports of name 'render'
const containerNode = useMemo(() => {
return containerRef ? containerRef.current : document.body;
}, [containerRef, state]);

Check warning on line 14 in packages/nidalee/src/Portal/Portal.tsx

GitHub Actions / build (18.x)

React Hook useMemo has an unnecessary dependency: 'state'. Either exclude it or remove the dependency array
useEffect(() => {
// on initial render when containerRef.current is a valid dom node