Skip to content

feat: Fix YAML syntax error in GitHub workflow#1

Open
seryl wants to merge 9 commits intomainfrom
claude/fix-rebuild-packages-yaml-011CUp7M7u8JkmYn5fUBhDUz
Open

feat: Fix YAML syntax error in GitHub workflow#1
seryl wants to merge 9 commits intomainfrom
claude/fix-rebuild-packages-yaml-011CUp7M7u8JkmYn5fUBhDUz

Conversation

@seryl
Copy link
Copy Markdown
Owner

@seryl seryl commented Nov 17, 2025

No description provided.

This workflow enables automatic regeneration of Nickel packages when:
- Triggered by the Amalgam repository via repository_dispatch
- Manually triggered via workflow_dispatch
- Weekly scheduled rebuild (Mondays at 2 AM UTC)

The workflow:
- Checks out the latest Amalgam code
- Builds Amalgam from source
- Regenerates all packages from .amalgam-manifest.toml
- Creates a PR with changes if any packages were updated

This integration allows nickel-pkgs to stay up-to-date with:
- Amalgam tooling improvements
- Upstream CRD changes
- Package metadata updates
Remove the automatic weekly rebuild schedule. The workflow will now
only be triggered by:
- repository_dispatch from the Amalgam repo
- manual workflow_dispatch
Update default fallback values to 'Manual trigger' and 'Manual'
since the scheduled trigger has been removed.
Use heredocs for multi-line strings in commit message and PR body
to avoid YAML parsing errors. This resolves the syntax error on line 90.
Replace heredoc syntax with simple variable assignments for commit
message and PR body to prevent YAML syntax errors.
Simplified multi-line commit message handling by:
- Removing problematic bash multi-line string variable
- Moving logic to compute MORE_FILES before commit
- Using direct git commit -m with properly formatted message
Changes to ensure YAML compatibility:
- Fixed heredoc indentation using spaces (not tabs) with sed cleanup
- Changed commit message from multi-line variable to printf with pipe
- Simplified PR creation to avoid complex multi-line strings
- Removed emojis and special characters from summary that confuse parsers
- All changes tested with Python YAML validator

This should now pass GitHub Actions YAML validation.
- Restored full PR body with trigger info, commit links, and checklist
- Removed emoji from header to ensure YAML compatibility
- Uses temp file with sed replacements for YAML-safe multiline content
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