Open
Conversation
Removes the root version since I can't properly gauge the impact and widespread use of AL easily. I also did this because the community version is simpler and more complete in my opinion.
Contributor
There was a problem hiding this comment.
Pull request overview
This PR removes the root-level AL.gitignore template, with the intention of relying on the existing Global/AL.gitignore instead.
Changes:
- Deleted the root
AL.gitignoretemplate (previously containing AL/Business Central project ignore rules).
Show a summary per file
| File | Description |
|---|---|
AL.gitignore |
Removes the root AL template entirely (eliminates AL project-specific ignore patterns). |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comments suppressed due to low confidence (1)
AL.gitignore:1
- Removing the root
AL.gitignoredrops several AL/Business Central-specific ignore rules (e.g.,.alcache/,.alpackages/,.output/,rad.json,*.g.xlf,*.flf,TestResults.xml) that are not covered byGlobal/AL.gitignore(which currently focuses on VS Code metadata). This would regress the AL template available in the root template chooser. If deduplication is the goal, consider keeping a rootAL.gitignore(per README folder structure) and moving editor-only rules to the appropriate Global template instead of removing AL-specific rules entirely.
- Files reviewed: 1/1 changed files
- Comments generated: 0
dooleydevin
approved these changes
Apr 17, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Link to the application or project's homepage
https://learn.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/devenv-programming-in-al
Reasons for making this change
Removes the root version since I can't properly gauge the impact and widespread use of AL easily. I also did this because the community version is simpler and more complete in my opinion.
Links to documentation supporting these rule changes
https://github.com/github/gitignore/blob/main/Global/AL.gitignore already exists and I think is more complete.
Unable to find proper justification for AL to live in root.
If this is a new template
NA
Merge and Approval Steps