search

Patrick-Kladek / Callisto

Parse Clang Static Analyzer messages and pass them to Slack
MIT License
21 stars 3 forks source link

Add posting to GitHub #5

Closed Patrick-Kladek closed 4 years ago

Patrick-Kladek commented 5 years ago

This allows Callisto to post the build Summary directly to GitHub. Posting to slack is not supported anymore.

Changes

Buildkite supports building both platforms simultaneously by splitting the workload to different servers. Because we don't want duplicate build warnings & errors Callisto now runs in two steps.

Step 1

Each build parses the output from fastlane and saves the information in a .buildreport file. This file is then uploaded to Buildkite because it's marked as build artifact/annotation - this is defined somewere in buildkite.yml

Step 2

After both builds finished (doesn't matter if on different or the same machine) the .buildreport files are read. This works because buildkite provides us the artifacts files - again defined somewhere in buildkite.yml. Callisto then filters duplicate build messages and splits them by platform. The resulting message is then posted (or updated if necessary) to GitHub as a comment.

schwmi commented 4 years ago

@Patrick-Kladek I think this one is already included in our CI setup - can we merge this also here into master?

Patrick-Kladek commented 4 years ago

@schwmi yes can be merged. Did you encounter any bugs until now? I also extend the PR Description. Does it make sense or do you need additional information?

schwmi commented 4 years ago

Thanks for the description 👍