Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(sage-monorepo): run Sonar on PRs that originates from this repo #2600

Conversation

tschaffter
Copy link
Member

@tschaffter tschaffter commented Mar 26, 2024

Closes #2599

UPDATE: The issue with adding the trigger pull_request is that it triggers another check, but we don't want to run twice the Sonar check.

Test

I will test this fix after merging to main by creating a same-repo PR.

Preview

Actually adding the trigger pull_request creates a new check that fails.

image

@tschaffter
Copy link
Member Author

See this comment. The current workflow should work fine so I'll close this PR.

@tschaffter tschaffter closed this Mar 27, 2024
@tschaffter tschaffter deleted the fix-sonar-workflow-internal-pr branch March 27, 2024 01:19
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.

[Bug] Run Sonar on PRs that originates from this repo
1 participant