[Excel] (Custom functions) Document unified manifest support#5596
[Excel] (Custom functions) Document unified manifest support#5596
Conversation
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit cea6ff7: ✅ Validation status: passed
For more details, please refer to the build report. |
|
Learn Build status updates of commit 1b47380: ✅ Validation status: passed
For more details, please refer to the build report. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit bf2dd40: ✅ Validation status: passed
For more details, please refer to the build report. |
|
Learn Build status updates of commit 4a80766: ❌ Validation status: errorsPlease follow instructions here which may help to resolve issue.
docs/excel/custom-functions-unified-manifest.md
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 1a8692f: ✅ Validation status: passed
For more details, please refer to the build report. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 9ddd838:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| docs/develop/configure-your-add-in-to-use-a-shared-runtime.md | View | Details | |
| docs/toc.yml | View | Details | |
| docs/develop/convert-xml-to-json-manifest.md | ✅Succeeded | View | |
| docs/excel/custom-functions-json.md | ✅Succeeded | View | |
| docs/excel/custom-functions-naming.md | ✅Succeeded | View | |
| docs/excel/custom-functions-overview.md | ✅Succeeded | View | |
| docs/excel/custom-functions-troubleshooting.md | ✅Succeeded | View | |
| docs/includes/excel-custom-functions-note.md | ✅Succeeded | n/a (file deleted or renamed) | |
| docs/quickstarts/excel-custom-functions-quickstart.md | ✅Succeeded | View |
docs/develop/configure-your-add-in-to-use-a-shared-runtime.md
- Line 25, Column 87: [Warning: file-not-found - See documentation]
Invalid file link: '../excel/custom-functions-unified-manifest.md'.
docs/toc.yml
- Line 713, Column 13: [Warning: file-not-found - See documentation]
Invalid file link: 'excel/custom-functions-unified-manifest.md'.
For more details, please refer to the build report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 5a6261f: ✅ Validation status: passed
For more details, please refer to the build report. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 807cdf9: ✅ Validation status: passed
For more details, please refer to the build report. |
There was a problem hiding this comment.
Pull request overview
Updates Excel custom functions documentation to describe unified manifest (Microsoft 365) support, replacing prior guidance that required the add-in-only (XML) manifest.
Changes:
- Adds unified-manifest instructions/examples across custom functions overview, naming, and JSON metadata articles.
- Removes the shared include note that stated unified manifest doesn’t support custom functions, and removes its usage from related pages.
- Refreshes article dates and makes minor wording/grammar edits.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| docs/quickstarts/excel-custom-functions-quickstart.md | Refreshes ms.date and clarifies a setup step’s wording. |
| docs/includes/excel-custom-functions-note.md | Removes the legacy note that blocked unified-manifest usage. |
| docs/excel/custom-functions-troubleshooting.md | Removes the legacy include note and improves intro/grammar (with one remaining grammar issue noted). |
| docs/excel/custom-functions-overview.md | Adds unified-manifest tabbed guidance for manifest configuration (tab formatting issues noted). |
| docs/excel/custom-functions-naming.md | Adds namespace configuration guidance for both manifest types via tabs. |
| docs/excel/custom-functions-json.md | Adds unified-manifest guidance for connecting metadataUrl via tabs (code fence issue noted). |
Comments suppressed due to low confidence (1)
docs/excel/custom-functions-troubleshooting.md:33
- Grammar/typo: "is also maybe a sign that that function" reads awkwardly and contains a duplicated "that". Rephrase to something like "may also be a sign that the function ..." and remove the duplicate word.
- A `#N/A` error is also maybe a sign that that function while registered could not be run. This is typically due to a missing `CustomFunctions.associate` command.
- A `#VALUE!` error typically indicates an error in the functions' script file.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Alison McKay <almckay@microsoft.com>
|
Learn Build status updates of commit 2226565:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| docs/excel/custom-functions-json.md | View | Details | |
| docs/excel/custom-functions-naming.md | ✅Succeeded | View | |
| docs/excel/custom-functions-overview.md | ✅Succeeded | View | |
| docs/excel/custom-functions-troubleshooting.md | ✅Succeeded | View | |
| docs/includes/excel-custom-functions-note.md | ✅Succeeded | n/a (file deleted or renamed) | |
| docs/quickstarts/excel-custom-functions-quickstart.md | ✅Succeeded | View |
docs/excel/custom-functions-json.md
- Line 66, Column 2: [Warning: disallowed-html-tag - See documentation]
HTML tag 'Resources' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.
For more details, please refer to the build report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit c9d9a72: ✅ Validation status: passed
For more details, please refer to the build report. |
|
Learn Build status updates of commit 607f2fb: ✅ Validation status: passed
For more details, please refer to the build report. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
This PR documents how to use the unified manifest for Microsoft 365 with custom functions add-ins. Once the unified manifest for custom functions is declared GA, this PR should be merged.