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

COMCL-168: Fix linters workflow #839

Merged
merged 2 commits into from Jan 14, 2022

Conversation

ahed-compucorp
Copy link

Overview

This PR fixes the failed linter workflow.

Screenshot from 2022-01-13 18-48-06

Technical Details

The Set Correct NodeJS version step bootstraps the project with nvm so that nvm will install Node.js 14.16.0 which is specified in the file .nvmrc.
The issue is the Run npm install step didn't use the version 14.16.0 and instead used the version 16.3.1.

Here, I introduce the setup-node action which allows us to specify a Node.js version. It was created by the Github team.

Before using the action, we read the the version specified in the file .nvmrc and set an output parameter so that the action can use it in the next step.

erawat
erawat previously approved these changes Jan 14, 2022
@ahed-compucorp ahed-compucorp changed the base branch from COMCL-167-searchkit-error to master January 14, 2022 10:52
@ahed-compucorp ahed-compucorp dismissed erawat’s stale review January 14, 2022 10:52

The base branch was changed.

erawat
erawat previously approved these changes Jan 14, 2022
@ahed-compucorp ahed-compucorp merged commit 7aec306 into master Jan 14, 2022
@ahed-compucorp ahed-compucorp deleted the COMCL-168-fix-linters-workflow branch January 14, 2022 11:07
This was referenced Jan 25, 2022
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