Skip to content

DYN-10279 Move markdown comment to the bottom#16966

Merged
QilongTang merged 1 commit into
DynamoDS:masterfrom
avidit:kca/dyn-10279
Mar 16, 2026
Merged

DYN-10279 Move markdown comment to the bottom#16966
QilongTang merged 1 commit into
DynamoDS:masterfrom
avidit:kca/dyn-10279

Conversation

@avidit
Copy link
Copy Markdown
Contributor

@avidit avidit commented Mar 16, 2026

Purpose

Move markdown comment to the bottom

Declarations

Check these if you believe they are true

Release Notes

N/A

Reviewers

(FILL ME IN) Reviewer 1 (If possible, assign the Reviewer for the PR)

(FILL ME IN, optional) Any additional notes to reviewers or testers.

FYIs

(FILL ME IN, Optional) Names of anyone else you wish to be notified of

Copilot AI review requested due to automatic review settings March 16, 2026 22:34
@avidit avidit changed the title Move markdown comment to the bottom DYN-10279 Move markdown comment to the bottom Mar 16, 2026
@avidit avidit requested review from QilongTang and Copilot and removed request for Copilot March 16, 2026 22:34
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

See the ticket for this pull request: https://jira.autodesk.com/browse/DYN-10279

Copy link
Copy Markdown
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 adjusts the generated GitHub agent wrapper markdown so the YAML frontmatter starts at the top of the file by moving the auto-generated HTML comment block to the bottom, and regenerates the affected wrappers accordingly.

Changes:

  • Update wrapper generation in sync_agent_wrappers.ps1 to emit the <!-- AUTO-GENERATED FILE ... --> marker at the end of the file.
  • Regenerate the six mapped .github/agents/*.agent.md wrapper files to match the new layout.

Reviewed changes

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

Show a summary per file
File Description
.github/scripts/sync_agent_wrappers.ps1 Moves the auto-generated marker block to the bottom in the generated wrapper template.
.github/agents/dynamo-unit-testing.agent.md Regenerated wrapper with marker moved to bottom.
.github/agents/dynamo-skill-writer.agent.md Regenerated wrapper with marker moved to bottom.
.github/agents/dynamo-pr-description.agent.md Regenerated wrapper with marker moved to bottom.
.github/agents/dynamo-onboarding.agent.md Regenerated wrapper with marker moved to bottom.
.github/agents/dynamo-jira-ticket.agent.md Regenerated wrapper with marker moved to bottom.
.github/agents/dynamo-dotnet-expert.agent.md Regenerated wrapper with marker moved to bottom.

You can also share your feedback on Copilot code review. Take the survey.

@sonarqubecloud
Copy link
Copy Markdown

@QilongTang
Copy link
Copy Markdown
Contributor

Thank you, I was curious why these wrappers did not work. Let's give it a try

@QilongTang QilongTang merged commit 6dd4cf6 into DynamoDS:master Mar 16, 2026
40 of 42 checks passed
@avidit avidit deleted the kca/dyn-10279 branch March 17, 2026 00:06
@QilongTang
Copy link
Copy Markdown
Contributor

@avidit It worked!!

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.

3 participants