Skip to content

Testing - #53

Closed
adityamandaleeka wants to merge 1 commit into
dotnet:mainfrom
adityamandaleeka:temp_debug2
Closed

Testing #53
adityamandaleeka wants to merge 1 commit into
dotnet:mainfrom
adityamandaleeka:temp_debug2

Conversation

@adityamandaleeka

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings February 20, 2026 03:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 a temporary debug step to the GitHub Actions evaluation workflow to investigate the location and presence of results.json files generated by the skill-validator tool.

Changes:

  • Added a debug step that prints the current working directory, finds all results.json files, and lists the contents of the .skill-validator-results directory

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

Comment on lines +90 to +96
- name: Debug results dir
if: always()
run: |
echo "cwd: $(pwd)"
find . -path './.git' -prune -o -name 'results.json' -print
ls -la .skill-validator-results/ 2>&1 || echo "dir not found"

Copilot AI Feb 20, 2026

Copy link

Choose a reason for hiding this comment

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

This debug step appears to be temporary troubleshooting code based on the PR title "Testing". Debug steps added for troubleshooting should be removed before merging to production. If this debugging information is valuable for future troubleshooting, consider making it conditional with a workflow input parameter or moving it to a separate debugging workflow.

Suggested change
- name: Debug results dir
if: always()
run: |
echo "cwd: $(pwd)"
find . -path './.git' -prune -o -name 'results.json' -print
ls -la .skill-validator-results/ 2>&1 || echo "dir not found"

Copilot uses AI. Check for mistakes.
moesac0970 pushed a commit to moesac0970/skills that referenced this pull request Jul 4, 2026
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.

2 participants