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

docs: Update CONTRIBUTING.md #367

Merged
merged 1 commit into from
Aug 4, 2023
Merged

docs: Update CONTRIBUTING.md #367

merged 1 commit into from
Aug 4, 2023

Conversation

doug-wade
Copy link
Owner

Add some extra documentation from what we've learned from onboarding new team members.

CONTRIBUTING.md Outdated

### Error: EEXIST: file already exists, mkdir 'path/to/tybalt/node_modules/aria-build/node_modules/esbuild/.install'

This can happen if the Nix process flakes. Remove all untracked files and reinstall the dependencies:
Copy link
Contributor

Choose a reason for hiding this comment

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

@doug-wade we can remove this particular error for now, as I am not really sure about why this particular error was triggered. And since not everyone uses Nix, this is a distro-specific issue. I'll make a simple flake.nix for other Nix users out there.

@doug-wade doug-wade force-pushed the update-contributing branch 2 times, most recently from 9587f7d to ad19457 Compare August 4, 2023 17:31
Add some extra documentation from what we've learned from onboarding new team members.
Copy link
Contributor

@Ashvith10 Ashvith10 left a comment

Choose a reason for hiding this comment

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

All looks fine to me.

@doug-wade doug-wade merged commit 23a243e into master Aug 4, 2023
4 checks passed
@doug-wade doug-wade deleted the update-contributing branch August 4, 2023 18:21
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

2 participants