Remove quotes from allowlist in CLA Assistant workflow#3
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR removes quotes from the allowlist parameter value in the CLA Assistant workflow configuration. While this change is technically valid YAML syntax and will function correctly, it introduces a style inconsistency with other string parameters in the same workflow file.
Key Changes:
- Modified the
allowlistparameter from quoted to unquoted format
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| # branch should not be protected | ||
| branch: "main" | ||
| allowlist: "semantic-release-bot,semantic-release[bot]" | ||
| allowlist: semantic-release-bot,semantic-release[bot] |
There was a problem hiding this comment.
[nitpick] Removing quotes from the allowlist parameter creates inconsistency with other string parameters in this workflow file. Other string parameters like branch: "main", remote-organization-name: "ctfpilot", and remote-repository-name: "cla" all use quotes. For consistency and maintainability, consider keeping the quotes around the allowlist value.
| allowlist: semantic-release-bot,semantic-release[bot] | |
| allowlist: "semantic-release-bot,semantic-release[bot]" |
1 participant
No description provided.