feat: add support for github split instructions#50
feat: add support for github split instructions#50DanielKlys7 wants to merge 3 commits intoprzeprogramowani:masterfrom
Conversation
| adaptableFileEnvironments.has(selectedEnvironment) && isMultiFileEnvironment; | ||
|
|
||
| const extension = | ||
| AIEnvironmentName.GitHub && isMultiFileEnvironment ? 'instructions.md' : 'mdc'; |
There was a problem hiding this comment.
Actually wondering whether it's a good solution or should we rather move it into some map for multiFileEnvironment paths for AIEnvironments
|
Thanks for your effort @DanielKlys7! With regards to the change, I believe we should always prefer good defaults over selectors and checkboxes. So if we believe that multi-file instructions are now considered default in Copilot, we should move from this: ...into something like this: Based on the documentation that you provided, we don't need to include config files. It is only required if you go against the convention of So:
Once this is done and we can keep this simple, we'll be able to accept this PR. (Another challenge is metadata - |


Description
Add support for github split instructions:
Nagranie.z.ekranu.2025-05-12.o.03.29.56.mov
Fix bug regarding stale isMultiFileEnvironment after reload:
Nagranie.z.ekranu.2025-05-12.o.02.31.38.mov
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Important Note for Fork-Based PRs
If you're submitting a PR from a forked repository, please note that E2E tests require specific repository-level environment (
integration) and secrets to be set up. These are described in.env.exampleand include:Please ensure these are properly configured in your fork's repository settings under "Secrets and variables" → "Actions" before running E2E tests.