Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fixing Yarn issue in CI #528

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

Merged
merged 4 commits into from
Nov 2, 2020
Merged

Fixing Yarn issue in CI #528

merged 4 commits into from
Nov 2, 2020

Conversation

alyezz
Copy link
Contributor

@alyezz alyezz commented Oct 12, 2020

  • Fixes an issue causing the CI to fail due to Yarn

@codecov
Copy link

codecov bot commented Oct 12, 2020

Codecov Report

Merging #528 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #528   +/-   ##
=======================================
  Coverage   89.23%   89.23%           
=======================================
  Files          21       21           
  Lines         381      381           
  Branches       72       72           
=======================================
  Hits          340      340           
  Misses         36       36           
  Partials        5        5           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7bc6c41...50bd774. Read the comment docs.

@alyezz alyezz changed the base branch from pr-template to master October 13, 2020 10:52
Copy link
Contributor

@AliAbdelfattah AliAbdelfattah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's probably not the best idea to ignore the node engine version incompatibilities. Although it's working now, it could easily break in the future.

Having a look at the release history for the library teeny-request which codecov depends on, I can see in v7 they've upgraded their node engine. Updating codecov from v3.7 to v3.8 which in turn updates teeny-request from v6 to v7 which causes incompatibility.

A temporary solution here until we decide to upgrade our CI node engine.

@AliAbdelfattah AliAbdelfattah merged commit 475d8ba into master Nov 2, 2020
@AliAbdelfattah AliAbdelfattah deleted the alyezz-patch-1 branch November 2, 2020 12:55
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.

2 participants