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

Improve dev-setup by generating a meta.json on setup #427

Merged
merged 2 commits into from
Nov 21, 2024

Conversation

rjbeers
Copy link
Contributor

@rjbeers rjbeers commented Nov 21, 2024

Summary

When a new dev tries to work on this repo, the node version isn't set correctly, causing yarn install issues, and the metadata file isn't generated, which makes things fail for unclear reasons. This PR addresses both.

Related issues

Fixes #425

metadata.json is required, so after building the CLI dependency, set the
matching metadata.json.
@rjbeers rjbeers added the bug Something isn't working label Nov 21, 2024
@rjbeers rjbeers requested review from a team as code owners November 21, 2024 16:10
@rjbeers rjbeers requested review from kenjenkins and removed request for a team November 21, 2024 16:10
@rjbeers rjbeers changed the title Improve dev-setup by generating a metadata.json on setup Improve dev-setup by generating a meta.json on setup Nov 21, 2024
Copy link
Collaborator

@nhayfield nhayfield left a comment

Choose a reason for hiding this comment

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

Lgtm

@rjbeers rjbeers merged commit 0551edb into main Nov 21, 2024
5 checks passed
@rjbeers rjbeers deleted the rj/improve-dev-setup branch November 21, 2024 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: src/meta.json doesn't exist
2 participants