Skip to content

Remove preinstall to see if that is causing issues with prerender-service #15

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 7 commits into from
Mar 20, 2023

Conversation

FSVetaz
Copy link
Contributor

@FSVetaz FSVetaz commented Mar 20, 2023

update @fs/eslint-config-frontier-react too, see fs-webdev/eslint-config-frontier-react#38

This is the issue in prerender-service, so seeing if this will fix this.
image

npm ERR! code 255
npm ERR! path /var/app/staging/node_modules/eslint-config-tree
npm ERR! command failed
npm ERR! command sh -c -- git config --global url.https://github.com/.insteadOf git://github.com/
npm ERR! warning: unable to access '/root/.gitconfig': Permission denied
npm ERR! warning: unable to access '/root/.config/git/config': Permission denied
npm ERR! error: could not lock config file /root/.gitconfig: Permission denied

@FSVetaz FSVetaz requested a review from a team as a code owner March 20, 2023 16:11
Copy link
Collaborator

@skye2k2 skye2k2 left a comment

Choose a reason for hiding this comment

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

Good job catching the ordering issue that we never saw locally because the files already existed from us running npm test previously.

I really would rather have a searchable text entry than a screenshot for the error we are attempting to resolve in the body of the commit, but this looks worth trying.

@FSVetaz FSVetaz merged commit 17954df into master Mar 20, 2023
@FSVetaz FSVetaz deleted the jordi/preinstall branch March 20, 2023 19:30
@skye2k2
Copy link
Collaborator

skye2k2 commented Mar 20, 2023

Meaning that you merging the pull request without modifying the commit message has me irritated.

@FSVetaz
Copy link
Contributor Author

FSVetaz commented Mar 20, 2023

Meaning that you merging the pull request without modifying the commit message has me irritated.

Oh! Sorry

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