Skip to content

Conversation

@jonasblunck
Copy link
Contributor

Add basic copilot instructions that makes it possible to ask Copilot about settings.

@jonasblunck jonasblunck marked this pull request as ready for review December 2, 2025 15:36
@jonasblunck jonasblunck requested a review from a team as a code owner December 2, 2025 15:36
Copilot AI review requested due to automatic review settings December 2, 2025 15:36
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds GitHub Copilot agent instructions to AL-Go templates, enabling users to ask Copilot about AL-Go settings and configuration. The implementation follows GitHub's convention for Copilot agent files by placing them in .github/.agents/ directories with the .agent.md extension.

Key Changes

  • Added copilot agent instruction files to both template types (AppSource App and Per Tenant Extension)
  • Instructions guide Copilot to reference the AL-Go settings documentation and release notes
  • Instructions specify that settings should be applied to AL-Go-Settings.json

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
Templates/AppSource App/.github/.agents/algo-settings.agent.md Copilot instructions for AppSource App template providing guidance on AL-Go settings
Templates/Per Tenant Extension/.github/.agents/algo-settings.agent.md Copilot instructions for Per Tenant Extension template providing guidance on AL-Go settings

Note: I identified a spelling error in line 7 of both files where "inforomation" should be "information", but I was unable to create inline comments as these files were not available in the PR diff context provided to me. The spelling error appears in: "For additional inforomation about AL-Go" and should be corrected to "For additional information about AL-Go".


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@aholstrup1 aholstrup1 enabled auto-merge (squash) December 5, 2025 13:23
@aholstrup1 aholstrup1 merged commit 18fe3d5 into microsoft:main Dec 5, 2025
6 checks passed
@aholstrup1 aholstrup1 mentioned this pull request Dec 8, 2025
4 tasks
aholstrup1 added a commit that referenced this pull request Dec 11, 2025
### ❔What, Why & How

<!-- Include description of the changes that will help reviewers in
their task -->

Add agents files to AL-Go sync

Follow up to #2010

### ✅ Checklist

- [ ] Add tests (E2E, unit tests)
- [ ] Update RELEASENOTES.md
- [ ] Update documentation (e.g. for new settings or scenarios)
- [ ] Add telemetry

<!-- Include more checklist entries, if needed -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants