Skip to content

[tidy] Format JSON schema file#2799

Closed
github-actions[bot] wants to merge 1 commit into
mainfrom
tidy/format-json-schema-eb4d3060aaa64296
Closed

[tidy] Format JSON schema file#2799
github-actions[bot] wants to merge 1 commit into
mainfrom
tidy/format-json-schema-eb4d3060aaa64296

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

Summary

This PR applies prettier formatting to the GitHub workflow JSON schema file for improved readability and consistency.

Changes Made

  • Formatted pkg/workflow/schemas/github-workflow.json:
    • Applied prettier formatting with single-line arrays
    • Removed unnecessary line breaks in array definitions
    • Maintained all functional schema definitions

What Was Fixed

The linting process detected unformatted JSON files. Running make fmt-json applied prettier formatting to ensure consistency with the project's code style guidelines.

Testing

  • make fmt - All Go code formatted correctly
  • make lint - All linting checks pass
  • make fmt-json - JSON files properly formatted
  • ⚠️ make test - Pre-existing test failures in TestStrictModeTimeout (unrelated to this change)

The test failures are related to missing permissions validation for GitHub MCP toolsets, which existed before this change. This PR only contains cosmetic JSON formatting changes and does not affect functionality.

Impact

  • Lines changed: 76 (9 insertions, 67 deletions)
  • Files affected: 1 (schema file only)
  • Breaking changes: None
  • Functional changes: None (formatting only)

Note: Workflow .lock.yml files were excluded from this PR as per tidying guidelines - they are compilation artifacts that should not be included in tidy commits.

AI generated by Tidy

- Applied prettier formatting to github-workflow.json
- Arrays now use single-line format for better readability
- No functional changes, only cosmetic formatting
@pelikhan pelikhan closed this Oct 30, 2025
@pelikhan pelikhan deleted the tidy/format-json-schema-eb4d3060aaa64296 branch November 4, 2025 23:00
@lpcox lpcox mentioned this pull request May 10, 2026
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant