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

README update for python error on yarn install #985

Merged
merged 1 commit into from Feb 12, 2022

Conversation

wd021
Copy link
Contributor

@wd021 wd021 commented Feb 10, 2022

Ran into an error #983 caused by python3 not being installed. Node-gyp v8.4.1 requires python3.7+.

Updated the readme to include this tidbit of info.

Breaking Change

Is this a breaking change? If yes, add notes below on why this is breaking and
what additional work is required, if any.

[ ] Yes
[x ] No

@NullSoldier NullSoldier changed the base branch from master to staging February 11, 2022 00:59
@NullSoldier
Copy link
Contributor

NullSoldier commented Feb 11, 2022

Thanks! Can you rebase this off of staging? We don't merge anything into master because people have scripts to auto pull, so our workflow is to merge everything into staging then merge it all into master from time to time once it's tested.

@wd021
Copy link
Contributor Author

wd021 commented Feb 11, 2022

understood, done!

Copy link
Member

@dguenther dguenther left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks!

@dguenther dguenther merged commit fee68e3 into iron-fish:staging Feb 12, 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

3 participants