Add environment variable support and update Docker configuration#5
Merged
ItsAlexIK merged 5 commits intoItsAlexIK:mainfrom Jan 2, 2026
Merged
Add environment variable support and update Docker configuration#5ItsAlexIK merged 5 commits intoItsAlexIK:mainfrom
ItsAlexIK merged 5 commits intoItsAlexIK:mainfrom
Conversation
Collaborator
Kf637
commented
Jan 2, 2026
- Moved ALLOWED_USER_ID and ALLOWED_ROLE_ID to .env.example for better configuration management.
- Updated htop.js to read user and role IDs from environment variables.
- Added Dockerfile for containerization with optimized dependency installation.
- Created .dockerignore to exclude unnecessary files from the Docker context.
- Moved ALLOWED_USER_ID and ALLOWED_ROLE_ID to .env.example for better configuration management. - Updated htop.js to read user and role IDs from environment variables. - Added Dockerfile for containerization with optimized dependency installation. - Created .dockerignore to exclude unnecessary files from the Docker context.
Collaborator
Author
ChangelogAll notable changes to this project are documented here. 2026-01-02Added
Changed
Fixed
|
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds containerization support and improves configuration management for a Discord bot by introducing environment variable support for user and role IDs, updating dependencies, and adding Docker deployment capabilities.
Key changes:
- Moved hardcoded user and role IDs to environment variables for better configuration management
- Updated multiple npm dependencies to newer versions
- Added Docker support with Dockerfile, .dockerignore, and GitHub Actions workflow for automated image publishing
Reviewed changes
Copilot reviewed 8 out of 11 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| src/commands/htop.js | Replaced hardcoded ALLOWED_USER_ID and ALLOWED_ROLE_ID with environment variable reads |
| .env.example | Added new environment variables for user and role IDs (with syntax issues) |
| package.json | Updated discord.js, dotenv, and systeminformation dependencies to newer versions |
| package-lock.json | Corresponding lockfile updates for dependency version changes |
| Dockerfile | New multi-stage Docker image configuration using Node.js 20 Alpine |
| .dockerignore | Excludes unnecessary files from Docker build context |
| .github/workflows/docker-publish.yml | Automated workflow to build and push Docker images to GHCR |
| index.js | Changed event listener from "ready" to "clientReady" and added logging statements |
| README.md | Updated asset paths from ./assets/ to .github/assets/ |
| .github/assets/status.png | Moved asset file to new location |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Owner
|
Thanks for pissing off Copilot |
Collaborator
Author
|
Np, always here to piss it off. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.