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

Fixing CI issues #127

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Fixing CI issues #127

wants to merge 1 commit into from

Conversation

jokkon
Copy link

@jokkon jokkon commented Sep 22, 2023

There are multiple issues relating to why the github actions build for the tests are failing:

  • Node12 is deprecated and github has moved from it to node16 for running github actions. It seems that this means that actions/chekout@v2 is deprecated for actions/checkout@v3, hence this change should fix all the tests.
  • It seems as if YubiKit is failing to load according to the logs, which states the likes of fatal error: module 'YubiKit' is not defined in any loaded module map file; maybe you need to load it. and umbrella header for module 'YubiKit' does not include header file.

The first seems fixed by the existing commit, but I am not sure how to move on having YubiKit loaded correctly.

Node12 is deprecated and github has moved from it to node16 for running github actions. It seems that this means that actions/chekout@v2 is deprecated for actions/checkout@v3, hence this change should fix all the tests.
@jokkon jokkon changed the title Changing checkout from node12 version Fixin CI issues Sep 22, 2023
@jokkon jokkon changed the title Fixin CI issues Fixing CI issues Sep 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant