Skip to content

Add docker-compose.override.yml to .gitignore#1137

Merged
daveearley merged 1 commit intoHiEventsDev:developfrom
officialJCReyes:patch-1
Mar 31, 2026
Merged

Add docker-compose.override.yml to .gitignore#1137
daveearley merged 1 commit intoHiEventsDev:developfrom
officialJCReyes:patch-1

Conversation

@officialJCReyes
Copy link
Copy Markdown
Contributor

Thank you for creating a PR! We appreciate your contribution to Hi.Events.

To make the process as smooth as possible, please ensure you've read the contributing guidelines before proceeding.

Please include a summary of the changes and the issue being fixed or the feature being added. The more detail, the better!

Checklist

  • I have read the contributing guidelines.
  • My code is of good quality and follows the coding standards of the project.
  • I have tested my changes, and they work as expected.

Thank you for your contribution! 🎉

Ignore the Docker Compose override file for local customizations
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 31, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@officialJCReyes
Copy link
Copy Markdown
Contributor Author

Added docker-compose.override.yml to .gitignore. This ensures that local configuration changes (e.g., custom ports, volume mounts, or environment variables) stay local and don't accidentally get committed to the repo.

Example: I added mem_limt: and cpu:, as well as changing my external port.

@officialJCReyes
Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@daveearley
Copy link
Copy Markdown
Contributor

Thanks, @officialJCReyes!

@daveearley daveearley merged commit d0980d5 into HiEventsDev:develop Mar 31, 2026
1 of 2 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Mar 31, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants