with-docker: add new config file formats#90731
Conversation
|
Allow CI Workflow Run
Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer |
|
Allow CI Workflow Run
Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer |
There was a problem hiding this comment.
Pull request overview
This PR updates the examples/with-docker Docker build context exclusions so the example ignores additional modern config file naming formats and Compose file extensions, keeping production images leaner.
Changes:
- Broadened test file ignores from explicit
*.test.{ts,tsx,js,jsx}/*.spec.{...}to*.test.*and*.spec.*, and added common test-runner config globs (Vitest/Jest/Cypress/Playwright). - Expanded ignored Docker/Compose filenames to include
compose.yml/.yamlanddocker-compose*.yaml. - Added ignores for newer tool config formats (e.g.
eslint.config.*,prettier.config.*,stylelint.config.*) and some AI agent metadata paths.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| .anthropic/ | ||
| .kiro | ||
| .claude | ||
| AGENTS.md |
There was a problem hiding this comment.
AGENTS.md is already excluded by the earlier *.md pattern, so this entry is redundant. Consider removing it (or, if the intent is to keep some markdown files in the build context, narrow the *.md rule and then keep explicit entries like this).
| AGENTS.md |
There was a problem hiding this comment.
Hm, even if it is redundant now, it would be nice to keep for future changes, and to signal that this file should be ignored
Co-authored-by: Karl Horky <karl.horky@gmail.com>
What?
Update config files in the
with-dockerexample inside the.dockerignoreWhy?
They are outdated
How?
Add lines
cc @icyJoseph