Skip to content

Dev to Main#47

Merged
gugu759153 merged 11 commits intomainfrom
dev
Mar 25, 2025
Merged

Dev to Main#47
gugu759153 merged 11 commits intomainfrom
dev

Conversation

@GXiang314
Copy link
Member

This pull request includes various changes to the project, focusing on Docker configuration, GitHub workflows, and backend/frontend setup. The most important changes include updating Docker configurations, adding GitHub workflows for building and testing, and modifying environment settings and package configurations.

Docker Configuration Updates:

  • Added .dockerignore files for the main project and apps/backend to exclude unnecessary files from Docker builds. [1] [2]
  • Updated Dockerfile for apps/backend to support multi-stage builds and improved dependency management.
  • Added Dockerfile for apps/frontend to support multi-stage builds and optimized dependencies.

GitHub Workflows:

  • Added workflow for building the web frontend service on push and pull request events.
  • Added workflow for building the heartbeat service on push and pull request events.
  • Added workflow for building and testing the backend service on push and pull request events.

Environment and Configuration Changes:

  • Updated environment variables in apps/backend/.env.development and apps/frontend/.env.development to reflect new port and host settings. [1] [2]
  • Modified package.json and tsconfig.json in apps/backend to reflect dependency updates and configuration changes for testing and building. [1] [2]
  • Added turbo.json to apps/backend for task management.

Code Changes:

  • Added signal handling in apps/backend/src/index.ts to gracefully shut down the server.
  • Updated import statements in apps/frontend/pages/room.vue to use lodash-es instead of lodash.

Miscellaneous:

  • Added .npmrc to enable linking workspace packages.
  • Updated GitHub Actions to use the latest versions of setup actions and pnpm. [1] [2] [3]

@GXiang314 GXiang314 requested a review from gugu759153 March 25, 2025 13:37
@GXiang314 GXiang314 self-assigned this Mar 25, 2025
@gugu759153 gugu759153 merged commit 3cab970 into main Mar 25, 2025
3 checks passed
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.

2 participants