Skip to content

impr: fully in docker, remove eslint#3

Merged
RyanPotat merged 2 commits intomainfrom
more-docker
Nov 2, 2025
Merged

impr: fully in docker, remove eslint#3
RyanPotat merged 2 commits intomainfrom
more-docker

Conversation

@RyanPotat
Copy link
Member

No description provided.

@RyanPotat RyanPotat requested a review from Copilot November 2, 2025 20:42
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes ESLint and Babel from the project and introduces Docker containerization with a multi-service setup. The changes streamline the build process by removing unused linting/transpilation tooling and add Docker infrastructure for deployment.

Key Changes:

  • Removed ESLint and Babel configurations and dependencies
  • Simplified build script (removed rm -rf ./dist)
  • Added Docker Compose setup with separate nginx and streamer services

Reviewed Changes

Copilot reviewed 7 out of 9 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
package.json Removed ESLint/Babel dependencies; simplified build script
package-lock.json Removed dependency tree for ESLint/Babel packages
eslint.config.mjs Deleted ESLint configuration file
babel.config.js Deleted Babel configuration file
streamer.example.conf Changed localhost to nginx hostname for Docker networking
docker-compose.yml Added multi-service Docker setup with streamer and nginx
Dockerfile.streamer New Dockerfile for the streamer application
Dockerfile.nginx No functional changes (whitespace only)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@RyanPotat RyanPotat merged commit 83ca455 into main Nov 2, 2025
2 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