Skip to content

Conversation

@petersg83
Copy link
Contributor

.strapi-updater.json was in the .gitignore of the example app but not in the .gitignore generated by generate-new. Causing issues in some environments : https://forum.strapi.io/t/error-open-packages-api-strapi-updater-json-permission-denied/1255/2

Some other variables were different between the 2 gitignores, I harmonized them.

@petersg83 petersg83 added issue: bug Issue reporting a bug source: cli Source is cli package labels Nov 20, 2020
@petersg83 petersg83 added this to the 3.3.4 milestone Nov 20, 2020
@codecov
Copy link

codecov bot commented Nov 20, 2020

Codecov Report

Merging #8707 (60e5647) into master (7dadfcc) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #8707   +/-   ##
=======================================
  Coverage   33.65%   33.65%           
=======================================
  Files        1248     1248           
  Lines       13765    13765           
  Branches     1365     1365           
=======================================
  Hits         4632     4632           
  Misses       8243     8243           
  Partials      890      890           
Flag Coverage Δ
front 25.33% <ø> (ø)
unit 54.36% <ø> (ø)

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


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 6b6c1e2...1cde72d. Read the comment docs.

Signed-off-by: Pierre Noël <petersg83@gmail.com>
@petersg83 petersg83 force-pushed the fix/update-notifier-gitignore branch from b668ace to 1cde72d Compare November 20, 2020 13:18
Copy link
Member

@derrickmehaffy derrickmehaffy 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
Copy link
Member

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

https://forum.strapi.io/t/error-open-packages-api-strapi-updater-json-permission-denied/1255/5

@alexandrebodin alexandrebodin merged commit ace487a into master Nov 25, 2020
@alexandrebodin alexandrebodin deleted the fix/update-notifier-gitignore branch November 25, 2020 08:54
andreamarchetti pushed a commit to andreamarchetti/strapi that referenced this pull request Dec 29, 2020
Signed-off-by: Pierre Noël <petersg83@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: cli Source is cli package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants