arthurnn / memcached

A Ruby interface to the libmemcached C client
Academic Free License v3.0
432 stars 125 forks source link

CI: Fix building for CodeQL Analysis #187

Closed dylanahsmith closed 3 years ago

dylanahsmith commented 3 years ago

Depends on https://github.com/arthurnn/memcached/pull/185 to fix test failures, see https://github.com/dylanahsmith/memcached/compare/test-fixes...fix-codeql-action for this PRs changes

Problem

The CodeQL Analysis Github Action on master is failing (https://github.com/arthurnn/memcached/actions/runs/692710994) with the error

We were unable to automatically build your code. Please replace the call to the autobuild action with your custom build steps.

Solution

As instructed, I added custom build steps (similar to https://github.com/arthurnn/memcached/pull/186) to build the native extension.

I'm not that familiar with how CodeQL hooks into the build process, so this might not analyze everything as desired, but it should at least be a step in the right direction.

dylanahsmith commented 3 years ago

@casperisfine for review