Skip to content

Conversation

@emersion
Copy link
Contributor

@emersion emersion commented Jun 5, 2025

We don't want to copy this over to the Docker daemon, since this directory can be quite large.

@AntoLC AntoLC added the docker label Jun 6, 2025
@AntoLC
Copy link
Collaborator

AntoLC commented Jun 6, 2025

LGTM!

Our commits must follow some rules, here some information about it: https://github.com/suitenumerique/docs/blob/main/CONTRIBUTING.md#commit-message-format

Contributors are required to sign off their commits with git commit --signoff: this confirms that they have read and accepted the Developer's Certificate of Origin 1.1. For security reasons we also require signing your commits with your SSH or GPG key with git commit -S.

Thank you for your contributions πŸŽ‰

@AntoLC AntoLC self-requested a review June 6, 2025 08:03
@emersion emersion changed the title Add .next to .dockerignore πŸ§‘β€πŸ’»(docker): add .next to .dockerignore Jun 8, 2025
@emersion emersion force-pushed the next-dockerignore branch from c5b417e to 9ab2ebf Compare June 8, 2025 15:51
@emersion
Copy link
Contributor Author

emersion commented Jun 8, 2025

Done.

@emersion emersion force-pushed the next-dockerignore branch from 9ab2ebf to 6019485 Compare June 8, 2025 15:52
We don't want to copy this over to the Docker daemon, since this directory can be quite large.
@lunika lunika force-pushed the next-dockerignore branch from 6019485 to 5ddc503 Compare June 11, 2025 06:15
@emersion
Copy link
Contributor Author

Oh, there is no colon in the commit title format. Will fix shortly.

@lunika
Copy link
Member

lunika commented Jun 11, 2025

Oh, there is no colon in the commit title format. Will fix shortly.

Yes and the gitmoji used is not the good one, should be πŸ™ˆ

@lunika
Copy link
Member

lunika commented Jun 11, 2025

Merged here bf66265

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 participants