Skip to content

Remove doubled 'the' in AppSourceCop rule descriptions#3567

Closed
jeffreybulanadi wants to merge 1 commit into
MicrosoftDocs:mainfrom
jeffreybulanadi:fix-doubled-words-appsourcecop-docs
Closed

Remove doubled 'the' in AppSourceCop rule descriptions#3567
jeffreybulanadi wants to merge 1 commit into
MicrosoftDocs:mainfrom
jeffreybulanadi:fix-doubled-words-appsourcecop-docs

Conversation

@jeffreybulanadi

Copy link
Copy Markdown
Contributor

Four AppSourceCop analyzer files contain 'the the' in rule description text. Fixes AS0080 ('mark the the original'), AS0128 ('removed from the the list'), AS0129 ('added to the the list'), and the summary table in appsourcecop.md (two occurrences).

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit bb243c1:

✅ Validation status: passed

File Status Preview URL Details
dev-itpro/developer/analyzers/appsourcecop-as0080.md ✅Succeeded
dev-itpro/developer/analyzers/appsourcecop-as0128.md ✅Succeeded
dev-itpro/developer/analyzers/appsourcecop-as0129.md ✅Succeeded
dev-itpro/developer/analyzers/appsourcecop.md ✅Succeeded

For more details, please refer to the build report.

@jeffreybulanadi jeffreybulanadi force-pushed the fix-doubled-words-appsourcecop-docs branch from bb243c1 to 2baa2e7 Compare May 15, 2026 00:08
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 2baa2e7:

✅ Validation status: passed

File Status Preview URL Details
dev-itpro/developer/analyzers/appsourcecop-as0080.md ✅Succeeded
dev-itpro/developer/analyzers/appsourcecop-as0128.md ✅Succeeded
dev-itpro/developer/analyzers/appsourcecop-as0129.md ✅Succeeded
dev-itpro/developer/analyzers/appsourcecop.md ✅Succeeded

For more details, please refer to the build report.

@SusanneWindfeldPedersen SusanneWindfeldPedersen added the do not merge yet This is waiting for an update, verification, or other stuff label May 19, 2026
@SusanneWindfeldPedersen

Copy link
Copy Markdown
Contributor

Thanks for your contribution. This PR can't be merged because it changes auto-generated content that comes from soure code - this content is placed inside DO-NOT-EDIT tags. Learn more here: https://learn.microsoft.com/en-us/dynamics365/business-central/dev-itpro/help/contributor-guide#autogenerated-content-in-the-developer-and-it-pro-content.

Hello - I see you have multiple PRs, so we will have to inspect them for this pattern and merge what we can. The rest must be fixed in source code and not in markdown. We'll keep them for now with a do not merge sign but we will close them eventually. Thanks.

@SusanneWindfeldPedersen

Copy link
Copy Markdown
Contributor

@jeffreybulanadi Thanks for your contribution. This PR can't be merged because it changes auto-generated content that comes from soure code - this content is placed inside DO-NOT-EDIT tags. Learn more here: https://learn.microsoft.com/en-us/dynamics365/business-central/dev-itpro/help/contributor-guide#autogenerated-content-in-the-developer-and-it-pro-content.

@SusanneWindfeldPedersen

Copy link
Copy Markdown
Contributor

Thanks for your contribution. This PR can't be merged because it changes auto-generated content that comes from source code — this content is placed inside DO_NOT_EDIT tags. Learn more here: https://learn.microsoft.com/en-us/dynamics365/business-central/dev-itpro/help/contributor-guide#autogenerated-content-in-the-developer-and-it-pro-content

@SusanneWindfeldPedersen

Copy link
Copy Markdown
Contributor

Thanks for spotting these issues. While the fixes are valid, this content is auto-generated from source code. We won't be updating the source for these particular changes as they don't meet the bar for a source code change at this time. We're closing this PR, but we appreciate your contribution.

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

Labels

do not merge yet This is waiting for an update, verification, or other stuff edits-auto-generated

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants