Skip to content

fix: prevent parantheses outside scope error#10

Merged
levivannoort merged 4 commits intomainfrom
task/prevent-parantheses-outside-scope-error
Apr 7, 2025
Merged

fix: prevent parantheses outside scope error#10
levivannoort merged 4 commits intomainfrom
task/prevent-parantheses-outside-scope-error

Conversation

@levivannoort
Copy link
Copy Markdown
Member

fixes: #9

@levivannoort levivannoort requested a review from Copilot April 7, 2025 19:45
@levivannoort levivannoort changed the title fix: prevent parantheses outside scope error fix: prevent parantheses outside (scope) error Apr 7, 2025
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (2)

main.go:105

  • [nitpick] Consider refining the error message for missing colon separation to include an example of the desired title format for clearer guidance.
if !found {

.github/workflows/update-contributors.yaml:33

  • Ensure that the workflow step is covered by tests to verify that the conditional check correctly skips the pull request process when there are no changes.
if: steps.check.outputs.changes == 'true'

@levivannoort levivannoort changed the title fix: prevent parantheses outside (scope) error fix: prevent parantheses outside scope error Apr 7, 2025
@levivannoort
Copy link
Copy Markdown
Member Author

output after changes:
{"time":"2025-04-07T19:46:02.29909062Z","level":"INFO","msg":"commit title validated successfully","type":"fix","scope":"","message":"prevent parantheses outside (scope) error"}

@levivannoort levivannoort merged commit 2ee51b3 into main Apr 7, 2025
1 check passed
@levivannoort levivannoort deleted the task/prevent-parantheses-outside-scope-error branch June 1, 2025 05:52
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.

False positive (error) using brackets in title

2 participants