-
-
Notifications
You must be signed in to change notification settings - Fork 3
npm install test 워크플로우 구현 #656
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
Conversation
|
@RiaOh 님, PR을 티켓과 연결 부탁드리겠습니다. 보드에 안 나와서 제가 계속 놓치고 있었네요. |
| @@ -0,0 +1,125 @@ | |||
| name: NPM Install Test - daleui | |||
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.
| name: NPM Install Test - daleui | |
| name: NPM Install Test |
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.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
DaleSeo
left a comment
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.
피드백 반영해주셔서 감사합니다.
91dfc80 to
1841324
Compare
Bundle ReportBundle size has no change ✅ |
|
@DaleSeo @hyoseong1994 아, 추가로 |
.github/workflows/integration.yml
Outdated
| # ======================================== | ||
| # 테스트 프로젝트 생성 + daleui 설치 | ||
| # ======================================== |
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.
이런 코멘트는 득보다 실이 많지 않을까요? 나중에 다른 분이 작업 내용을 수정할 때 함께 업데이트하는 것을 까먹기 쉽거든요. name 필드에서 작업 내용을 영어로 잘 나타내고 있기 때문에 불필요한 코멘트라고 생각합니다.
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.
@DaleSeo 아하, 네엡! 알겠습니다. 코멘트들은 제거하겠습니다!
.github/workflows/integration.yml
Outdated
| - name: "Create React App (React 18)" | ||
| project: cra |
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.
@RiaOh 님, 우리 CRA 테스트는 그냥 하지 마시죠. CRA는 2022년 4월 이후로 업데이트가 전혀 없으며, React 공식 문서에서도 사라지면서 사실 상 폐기(deprecated) 수순을 밟고 있거든요. 워크플로우 로그를 보시면 high 단계 보안 취약점이 6개나 있는 걸 보면, 이 프로젝트 사실 상 제대로 유지보수가 되고 있다고 보기 어려울 것 같습니다. 2026년에는 React 생태계가 CRA에서 완전히 벗어나는 동참하는 차원에서 우리도 쓰지 말고, 사용자들도 쓰지 말라고 권장했으면 좋겠습니다.
CRA 제거하면 matrix 안 써도 되서 워크플로우도 간단해지고, 1분 내에 실행이 끝날 수 있을 것입니다. 워크플로우가 이제 PR이 병합될 때가 아니라 PR에 변경이 있을 때 마다 실행이 되기 때문에 빠른 피드백을 줄 수 있어서 개발자 경험에도 도움이 될 거에요.
맞아요 Box는 최근에 업데이트된거라서 index에 추가안된거같네요 |
hyoseong1994
left a comment
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.
CRA 제거하여 vite만 테스트하는 부분에 대해서 동의합니다.
이미 리뷰가 달려있어서 해당부분만 수정해주시면 좋을거같아요!
| runs-on: ubuntu-latest | ||
| name: "Vite + React" | ||
|
|
||
| steps: | ||
| - name: Checkout repository | ||
| uses: actions/checkout@v6 | ||
|
|
||
| - name: Setup Bun | ||
| uses: oven-sh/setup-bun@v2 | ||
| with: | ||
| bun-version: latest | ||
|
|
||
| - name: Install dependencies & Build | ||
| run: | | ||
| bun install | ||
| bun run build:lib | ||
| - name: Create test project & install daleui | ||
| run: | | ||
| mkdir /tmp/test-project && cd /tmp/test-project | ||
| bun create vite@5 . --template react-ts | ||
| bun install | ||
| bun add "$GITHUB_WORKSPACE" | ||
| - name: Add daleui smoke test | ||
| run: | | ||
| cd /tmp/test-project | ||
| mkdir -p src/smoke | ||
| cat > src/smoke/DaleTest.tsx << 'EOF' | ||
| import { Button } from 'daleui'; | ||
| export default function DaleTest() { | ||
| return ( | ||
| <div data-testid="daleui-smoke"> | ||
| <Button>daleui 성공적으로 로드됨!</Button> | ||
| </div> | ||
| ); | ||
| } | ||
| EOF | ||
| - name: Run production build | ||
| run: | | ||
| cd /tmp/test-project | ||
| bun run build |
Check warning
Code scanning / CodeQL
Workflow does not contain permissions Medium
Bumps [eslint-plugin-testing-library](https://github.com/testing-library/eslint-plugin-testing-library) from 7.13.5 to 7.13.6. - [Release notes](https://github.com/testing-library/eslint-plugin-testing-library/releases) - [Changelog](https://github.com/testing-library/eslint-plugin-testing-library/blob/main/release.config.js) - [Commits](testing-library/eslint-plugin-testing-library@v7.13.5...v7.13.6) --- updated-dependencies: - dependency-name: eslint-plugin-testing-library dependency-version: 7.13.6 dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps the storybook group with 6 updates: | Package | From | To | | --- | --- | --- | | [@storybook/addon-a11y](https://github.com/storybookjs/storybook/tree/HEAD/code/addons/a11y) | `10.1.10` | `10.1.11` | | [@storybook/addon-designs](https://github.com/storybookjs/addon-designs) | `11.1.0` | `11.1.1` | | [@storybook/addon-docs](https://github.com/storybookjs/storybook/tree/HEAD/code/addons/docs) | `10.1.10` | `10.1.11` | | [@storybook/addon-themes](https://github.com/storybookjs/storybook/tree/HEAD/code/addons/themes) | `10.1.10` | `10.1.11` | | [@storybook/react-vite](https://github.com/storybookjs/storybook/tree/HEAD/code/frameworks/react-vite) | `10.1.10` | `10.1.11` | | [storybook](https://github.com/storybookjs/storybook/tree/HEAD/code/core) | `10.1.10` | `10.1.11` | Updates `@storybook/addon-a11y` from 10.1.10 to 10.1.11 - [Release notes](https://github.com/storybookjs/storybook/releases) - [Changelog](https://github.com/storybookjs/storybook/blob/next/CHANGELOG.md) - [Commits](https://github.com/storybookjs/storybook/commits/v10.1.11/code/addons/a11y) Updates `@storybook/addon-designs` from 11.1.0 to 11.1.1 - [Release notes](https://github.com/storybookjs/addon-designs/releases) - [Changelog](https://github.com/storybookjs/addon-designs/blob/master/CHANGELOG.v1-6.md) - [Commits](storybookjs/addon-designs@v11.1.0...v11.1.1) Updates `@storybook/addon-docs` from 10.1.10 to 10.1.11 - [Release notes](https://github.com/storybookjs/storybook/releases) - [Changelog](https://github.com/storybookjs/storybook/blob/next/CHANGELOG.md) - [Commits](https://github.com/storybookjs/storybook/commits/v10.1.11/code/addons/docs) Updates `@storybook/addon-themes` from 10.1.10 to 10.1.11 - [Release notes](https://github.com/storybookjs/storybook/releases) - [Changelog](https://github.com/storybookjs/storybook/blob/next/CHANGELOG.md) - [Commits](https://github.com/storybookjs/storybook/commits/v10.1.11/code/addons/themes) Updates `@storybook/react-vite` from 10.1.10 to 10.1.11 - [Release notes](https://github.com/storybookjs/storybook/releases) - [Changelog](https://github.com/storybookjs/storybook/blob/next/CHANGELOG.md) - [Commits](https://github.com/storybookjs/storybook/commits/v10.1.11/code/frameworks/react-vite) Updates `storybook` from 10.1.10 to 10.1.11 - [Release notes](https://github.com/storybookjs/storybook/releases) - [Changelog](https://github.com/storybookjs/storybook/blob/next/CHANGELOG.md) - [Commits](https://github.com/storybookjs/storybook/commits/v10.1.11/code/core) --- updated-dependencies: - dependency-name: "@storybook/addon-a11y" dependency-version: 10.1.11 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: storybook - dependency-name: "@storybook/addon-designs" dependency-version: 11.1.1 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: storybook - dependency-name: "@storybook/addon-docs" dependency-version: 10.1.11 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: storybook - dependency-name: "@storybook/addon-themes" dependency-version: 10.1.11 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: storybook - dependency-name: "@storybook/react-vite" dependency-version: 10.1.11 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: storybook - dependency-name: storybook dependency-version: 10.1.11 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: storybook ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [eslint-plugin-storybook](https://github.com/storybookjs/storybook/tree/HEAD/code/lib/eslint-plugin) from 10.1.9 to 10.1.11. - [Release notes](https://github.com/storybookjs/storybook/releases) - [Changelog](https://github.com/storybookjs/storybook/blob/next/CHANGELOG.md) - [Commits](https://github.com/storybookjs/storybook/commits/v10.1.11/code/lib/eslint-plugin) --- updated-dependencies: - dependency-name: eslint-plugin-storybook dependency-version: 10.1.11 dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>

변경 사항
목적
리뷰어에게
PR 작성자 체크 리스트