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

npm 배포 설정 #9

Merged
merged 5 commits into from
Jan 31, 2024
Merged

npm 배포 설정 #9

merged 5 commits into from
Jan 31, 2024

Conversation

Hanna922
Copy link
Member

@Hanna922 Hanna922 commented Jan 31, 2024

1️⃣ 어떤 작업을 했나요? (Summary)

  • package.json, vite.config.js 수정
  • npmignore 생성
  • vite-plugin-dts 설치
  • resolved npm 배포 설정 #8

(+ 코드오너 파일 같이 수정했습니다)

4️⃣ 체크리스트 (Checklist)

  • main 브랜치의 최신 코드를 pull 받았나요?

@Hanna922 Hanna922 self-assigned this Jan 31, 2024
"type": "git",
"url": "git+https://github.com/yourssu/YLS-Web.git"
},
"author": "",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

큰건 아니구! 이거 없어도 되나용? ㅋㅎㅋㅎ

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

앜ㅋㅋㅋㅋㅋㅋ 없어도 괜찮아요!! 저희는 author가 좀 유동적이라 생각해서 작성하지 않았슴다 ㅎ

@@ -39,7 +57,7 @@
"react-dom": "^18.2.0",
"react-router-dom": "^6.21.3",
"typescript": "^5.2.2",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ts 관련한 의존성은 devDependencies로 빼는거 어떨까요? 어차피 soomsilV2에 있고, 개발과정에서만 필요한것 같아서용!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

엇 이미 devDependencies에 있는데 혹쉬 다르게 보이시나요.....????

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

헉 죄송해요 그 펼치는거 안해서! 잘못봤네영! 머지하셔도 될것 같아용! 수고하셨습니당!!

@@ -39,7 +57,7 @@
"react-dom": "^18.2.0",
"react-router-dom": "^6.21.3",
"typescript": "^5.2.2",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

헉 죄송해요 그 펼치는거 안해서! 잘못봤네영! 머지하셔도 될것 같아용! 수고하셨습니당!!

@Hanna922 Hanna922 merged commit 49c76a1 into develop Jan 31, 2024
@Hanna922 Hanna922 deleted the feat/#8 branch January 31, 2024 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

npm 배포 설정
2 participants