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 github CI clang-format related check_c-cpp job #1752

Merged
merged 2 commits into from
Jul 21, 2023
Merged

Conversation

ia
Copy link
Collaborator

@ia ia commented Jul 21, 2023

  • Please check if the PR fulfills these requirements
  • The changes have been tested locally
  • There are no breaking changes
  • What kind of change does this PR introduce?
    Fix github CI clang-format related check_c-cpp job.

  • What is the current behavior?
    After check-style logic has been moved to shell script, make check-style calls it but because bash is missing in the container, script can't run the check!

  • What is the new behavior (if this is a feature change)?
    It should be fixed with these changes.

  • Other information:
    I'm terribly sorry!!! But it seems this particular job is cursed! It's the second time already when according to log there is a problem but github CI just can't track down such ease-to-automate checks for users. Not excusing my own responsibility here though! (going to a chalk board) I will be checking build steps' logs on github after changing scripts for build steps. I will be checking build steps' logs on github after changing scripts for build steps. I will be ... (not copy pasted BTW)

@ia
Copy link
Collaborator Author

ia commented Jul 21, 2023

Now it's shellcheck little report, hold on, please...

@ia
Copy link
Collaborator Author

ia commented Jul 21, 2023

Now it's shellcheck little report, hold on, please...

Ok, now it seems good to go.

@Ralim Ralim merged commit 52dd27c into Ralim:dev Jul 21, 2023
15 checks passed
@ia ia deleted the fix-clang-job branch July 23, 2023 21:31
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.

None yet

2 participants