Skip to content

fix(ci): quote publish.yml if-expression and clean up competitive analysis#28

Merged
carlos-alm merged 2 commits into
mainfrom
fix/publish-yaml-syntax
Feb 23, 2026
Merged

fix(ci): quote publish.yml if-expression and clean up competitive analysis#28
carlos-alm merged 2 commits into
mainfrom
fix/publish-yaml-syntax

Conversation

@carlos-alm

Copy link
Copy Markdown
Contributor

Summary

  • Fix YAML parse error in publish.yml by wrapping the preflight job's if expression in double quotes — single-quoted strings inside the expression ('release', 'chore: release v') were being interpreted as YAML syntax
  • Remove the "Irrelevant Repos" section (55 entries) from COMPETITIVE_ANALYSIS.md

Test plan

  • Verify publish.yml passes GitHub Actions YAML validation
  • Confirm workflow triggers correctly on push to main

Add deps:tree script that runs npm ls --all and writes the full
dependency tree to DEPENDENCIES.md. Hooked into prepare so it
stays up to date whenever dependencies change.
Wrap the preflight job's if-expression in double quotes to fix YAML
parse error caused by single-quoted strings inside the expression.
Remove the "Irrelevant Repos" section from COMPETITIVE_ANALYSIS.md.
Fix line-ending formatting in src/ and tests/.
@greptile-apps

greptile-apps Bot commented Feb 23, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Fixed YAML parse error in publish.yml by wrapping the if expression in double quotes, and cleaned up COMPETITIVE_ANALYSIS.md by removing the 55-entry "Irrelevant Repos" section.

  • The YAML fix addresses a parse error where single-quoted strings ('release', 'chore: release v') inside the expression were being misinterpreted as YAML syntax
  • The documentation cleanup improves readability by removing non-competitive repositories from the analysis

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • The changes are straightforward: a YAML syntax fix that resolves a documented parse error, and documentation cleanup that removes irrelevant content. No functional code changes or logic modifications.
  • No files require special attention

Important Files Changed

Filename Overview
.github/workflows/publish.yml Fixed YAML parse error by wrapping if-expression in double quotes
COMPETITIVE_ANALYSIS.md Removed 55-entry "Irrelevant Repos" section for cleaner documentation

Last reviewed commit: 580a614

@greptile-apps greptile-apps Bot 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.

4 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@carlos-alm carlos-alm merged commit 168a056 into main Feb 23, 2026
18 checks passed
@carlos-alm carlos-alm deleted the fix/publish-yaml-syntax branch February 23, 2026 01:15
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.

1 participant