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

[feature/build-and-analyze] add build & analyze GitHub workflow #1411

Merged
merged 11 commits into from
Nov 8, 2024

Conversation

felix-schwarz
Copy link
Contributor

Description

Adds build & analyze GitHub workflow.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@felix-schwarz felix-schwarz self-assigned this Oct 22, 2024
@DeepDiver1975
Copy link
Member

image

@DeepDiver1975
Copy link
Member

I see no real build in the output .... did I miss anything? THX

@felix-schwarz
Copy link
Contributor Author

@DeepDiver1975 The script performs two operations:

  1. perform a regular build of the target. You can see the last steps running and eventually the complete build process succeeding here:
    Bildschirmfoto 2024-11-08 um 08 30 28

  2. run the static analyzer

The latter is not strictly needed, but having the results of it ready might help during code review to quickly see if there are any relevant static analyzer findings without having to perform that time consuming step locally.

@DeepDiver1975
Copy link
Member

I totally missed this when parsing the log - sorry

@DeepDiver1975 DeepDiver1975 merged commit ce78dc8 into master Nov 8, 2024
4 of 5 checks passed
@DeepDiver1975 DeepDiver1975 deleted the feature/build-and-analyze branch November 8, 2024 08:38
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.

3 participants