-
Notifications
You must be signed in to change notification settings - Fork 7
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
Easy way to use forked packages #50
Comments
Interesting. Didn't know about But what I am concerned with is working together with others. I cannot ask everyone in a team to setup |
?? @mitar you've been here forever and didnt know about |
:-))) That's why it is great to have amazing people around me to teach me new things every day. :-) I love |
I see, as you're pointing out, the thing is that now you can have as for the But the easiest thing might be using a clone of the package repository just under There's an open issue about spreading this techniques a bit more directly on atmosphere, but I agree that a more official way to have an more direct way to test modification to packages might improve the workflow of many developers. +1 |
And one has to put package repository under |
whatever name you prefer for the folder! I'm quite sure meteor is doing a pre-scan of all folders under |
See my comment here. |
I think forking packages is already on the mdg todo? |
Actually I think the problem is only for core package that are used as a dependency of other core packages (like |
It is, if |
Currently there is no easy way to operate from a GitHub fork of a package. So imagine that I would have a package and would like to modify it a bit, just to see if it works better/fixes the problem (but still collaborate with other team members through git). I cannot make a fork on GitHub and link that as a dependency. I can only make a submodule in
packages
directory. But then, if package defines package name, or uses that anywhere as a dependency (for tests), I have to rename the package as well everywhere in the code. This is cumbersome and require unnecessary changes to the package repository.The text was updated successfully, but these errors were encountered: