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

How to contribute? #20

Closed
serkandurusoy opened this issue Jun 5, 2016 · 4 comments
Closed

How to contribute? #20

serkandurusoy opened this issue Jun 5, 2016 · 4 comments
Assignees
Labels
Type: Feature New features and feature requests

Comments

@serkandurusoy
Copy link
Contributor

The project requires a contribution guide that describes how to

  • set up necessary environment
  • development dependencies and how to install them (lerna and stuff)

And then we should describe

  • hot contribution workflow should be
  • base coding style guidelines
  • things to watch out for
  • things to do etc
@radekmie
Copy link
Contributor

radekmie commented Jun 5, 2016

I think I will create a CONTRIBUTING.md - it's a clean way to do it.

@radekmie radekmie self-assigned this Jun 5, 2016
@radekmie radekmie added the Type: Feature New features and feature requests label Jun 5, 2016
@serkandurusoy
Copy link
Contributor Author

Thank you

@jakobloekke
Copy link

How do I include the local, forked version in my main project?
I tried the usual npm link in uniforms and uniforms-semantic, but I can't get it to work. I have no experience with Lerna-projects – what steps are necessary to prepare the project for use?

Here's the error I get:

Unable to resolve some modules:

  "/Users/jakobloekkemadsen/web/forks/uniforms/packages/uniforms" in
/Users/jakobloekkemadsen/web/projects/name_of_meteor_project/node_modules/uniforms-semantic/node_modules/uniforms/index.js
(os.osx.x86_64)

It feels like some cross-referencing is not set up properly.

@radekmie
Copy link
Contributor

@jakobloekke
Unluckily, there's no easy way. You can use relative path in package.json, and then refresh it with:

rm -rf node_modules/uniforms* && npm i

@Monteth Monteth added this to Closed in Open Source (migrated) Jul 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature New features and feature requests
Projects
Archived in project
Development

No branches or pull requests

3 participants