skorotkov commented on PR #11721: URL: https://github.com/apache/ignite/pull/11721#issuecomment-2530220260
Hi Arnout Engelen, Thanks! Now it does the build. But we still need to return back the fetch of the PR's base branch (which is master in fact) from the upstream (main ignite) repo. Otherwise the Sonar would fail to detect the "new lines of code" if fork's master is not synchronized with main repo. See details at https://community.sonarsource.com/t/how-to-use-sonarcloud-with-a-forked-repository-on-github/7363/32 Do you see any security risks in fetching of the master branch from the main ignite repo? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@ignite.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
