Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 풀 리퀘스트는 Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
There was a problem hiding this comment.
Code Review
이 PR은 커스텀 도메인 설정을 위해 CNAME 파일을 추가합니다. 커스텀 도메인 적용을 위한 올바른 첫 단계입니다.
하지만 이 변경만으로는 커스텀 도메인에서 사이트가 올바르게 동작하지 않을 수 있습니다. web-io-game/vite.config.ts 파일의 base 설정이 '/Beyondworm/'으로 되어 있어, 커스텀 도메인 루트에서 에셋(CSS, JS 등)을 불러오지 못하는 문제가 발생할 것입니다. 커스텀 도메인을 사용하려면 base 설정을 '/'로 변경해야 합니다. 이 변경 사항을 현재 PR에 함께 포함하는 것을 강력히 권장합니다.
|
Custom Domain을 사용하기위해 vite config의 base경로를 추가로 수정했습니다. github pages의 기본 경로를 사용하게되면 beyond-imagination.github.io/Beyondworm/ 이기 때문에 vite빌드에서 base 경로에 /Beyondworm/을 붙여야 했습니다. (웹 브라우저의 루트는 beyond-imagination.github.io/)(프로젝트 명이 자동으로 붙음) Custom Domain으로 변경하게되면 **beyondwormgame.beyond-imagination.net/**가 루트 경로가 되기 때문에, vite빌드의 base 경로를 /로 지정해줘야 beyondwormgame.beyond-imagination.net/assets/main.js 로 제대로 에셋 결과물을 요청할 수 있습니다. |
| @@ -0,0 +1 @@ | |||
| beyondwormgame.beyond-imagination.net | |||
There was a problem hiding this comment.
game 이라는 문구는 때도 괜찮을것 같아요!
There was a problem hiding this comment.
아 beyondworm.beyond-imagination.net 은 이미 백엔드에서 사용하고있는 도메인이라 game을 붙였습니다 ㅎㅎㅎ;;
No description provided.