Skip to content

Conversation

@derrickmehaffy
Copy link
Member

Signed-off-by: Derrick Mehaffy derrickmehaffy@gmail.com

What does it do?

Projects were being generated with the node engine >=10.0.0 and it was causing issues deploying on heroku which watches this to figure out which version of node to install

Why is it needed?

We don't support node v15

Related issue(s)/PR(s)

https://forum.strapi.io/t/how-do-i-connect-heroku-to-strapi/703/5

Signed-off-by: Derrick Mehaffy <derrickmehaffy@gmail.com>
@derrickmehaffy derrickmehaffy added issue: bug Issue reporting a bug source: core:strapi Source is core/strapi package labels Oct 27, 2020
@derrickmehaffy derrickmehaffy added this to the 3.2.6 milestone Oct 27, 2020
@derrickmehaffy derrickmehaffy requested a review from a team October 27, 2020 16:17
@lauriejim
Copy link
Contributor

This pull request has been mentioned on Strapi Community. There might be relevant details there:

https://forum.strapi.io/t/how-do-i-connect-heroku-to-strapi/703/8

@codecov
Copy link

codecov bot commented Oct 27, 2020

Codecov Report

Merging #8497 into master will decrease coverage by 0.00%.
The diff coverage is 37.50%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #8497      +/-   ##
==========================================
- Coverage   33.26%   33.25%   -0.01%     
==========================================
  Files        1221     1221              
  Lines       13629    13635       +6     
  Branches     1357     1359       +2     
==========================================
+ Hits         4534     4535       +1     
- Misses       8211     8216       +5     
  Partials      884      884              
Flag Coverage Δ
#front 24.71% <ø> (ø)
#unit 54.63% <37.50%> (-0.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
packages/strapi-utils/lib/models.js 24.69% <37.50%> (-0.31%) ⬇️

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 dc0a91c...3a668f4. Read the comment docs.

Copy link
Member

@alexandrebodin alexandrebodin left a comment

Choose a reason for hiding this comment

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

LGTM

@derrickmehaffy derrickmehaffy merged commit 717a834 into master Oct 29, 2020
@derrickmehaffy derrickmehaffy deleted the fix/nodeEngine branch October 29, 2020 14:48
pascalvaccaro pushed a commit to pascalvaccaro/strapi that referenced this pull request Oct 29, 2020
Signed-off-by: Derrick Mehaffy <derrickmehaffy@gmail.com>
@petersg83 petersg83 modified the milestones: 3.2.6, 3.3.0 Nov 9, 2020
hdeadman pushed a commit to hdeadman/strapi that referenced this pull request Nov 12, 2020
Signed-off-by: Derrick Mehaffy <derrickmehaffy@gmail.com>
petersg83 pushed a commit that referenced this pull request Nov 13, 2020
Signed-off-by: Derrick Mehaffy <derrickmehaffy@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

issue: bug Issue reporting a bug source: core:strapi Source is core/strapi package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants