Jump to content

Help Contributing to Babylonjs


DylanD
 Share

Recommended Posts

Hello everyone, 

I am very inexperienced with git and open source project, and have recently gotten the time and the OK to contribute a few modules to Babylonjs.  However after following https://doc.babylonjs.com/how_to/how_to_start i have a few problems.

The first being gulp (which i dont believe i need as i have tested both modules, and i think that was the intention with gulp).  When i tried to follow along using the gulp command in the gulp folder produced this error.

 

Anyway i skipped that and have gotten to the point where i have commited my file and wish to make a pull request to babylonjs (master?) repository.  However i dont fully understand how to do that as it does not go over it in the tutorial.  So here is what i tried

Any ideas on what im doing wrong?

gulpHelp.PNG

Git diff.PNG

Link to comment
Share on other sites

Hello ! Welcome to our awesome community :) 
you need to do npm install first, that should solve the- 'cannot find module gulp-uglify'

if continues, do a manual - `npm install  gulp-uglify`

For guidelines - https://github.com/BabylonJS/Babylon.js/blob/master/contributing.md
For setup - http://doc.babylonjs.com/how_to/how_to_start

Link to comment
Share on other sites

@DylanD glad to hear that you are going to make a contribution. Even though I have had a couple of PRs accepted I still have problems when I do it usually with gulp. I think I will see if I can do a re-write that takes people who know little about git and github (as I was once and still no expert) on a step by step journey through it. I may be wrong but I think you are missing a repository on github and a fork of babylon.js in your repository. You can only issue a pull request from your fork to  that of babylon.js.

More here on forks

Having made and tested your local version you then need to check your version is built on the latest version of the babylon.js repository by doing a pull from that repository and then doing a push to your repository before issuing a pull request.

Pull and Pull Request difference

If you use git pull, you pull the changes from the remote repository into yours.

If you send a pull request to another repository, you ask their maintainers to pull your changes into theirs (you more or less ask them to use a git pull from your repository). From stackoverflow

 

Link to comment
Share on other sites

I know how to use git/github and I had got the same issue when building BabylonJS with gulp.

You can't really skip this step because you should no make a pull request if you did not test your code after building BabylonJS :)

@Deltakosh told me to check that I installed gulp 4.0.0 but I did not have time to try.

To know more about the git workflow usually used in open source project, see fork workflow in this documentation https://www.atlassian.com/git/tutorials/comparing-workflows. Maybe read about the simpler workflows first. Note that Bitbucket is just an alternative to Github, everything they say is applicable to Github.

Link to comment
Share on other sites

6 hours ago, Nodragem said:

I know how to use git/github and I had got the same issue when building BabylonJS with gulp.

You can't really skip this step because you should no make a pull request if you did not test your code after building BabylonJS :)

@Deltakosh told me to check that I installed gulp 4.0.0 but I did not have time to try.

To know more about the git workflow usually used in open source project, see fork workflow in this documentation https://www.atlassian.com/git/tutorials/comparing-workflows. Maybe read about the simpler workflows first. Note that Bitbucket is just an alternative to Github, everything they say is applicable to Github.

 

I use bitbucket for work and school.  So i am a bit familiar for it.

Link to comment
Share on other sites

So I followed the steps from there: https://doc.babylonjs.com/how_to/how_to_start as you did and everything went perfectly fine. I did not get the same error as before.

Note that if this screenshot describes what you did:

gulpHelp.PNG

Then it seems that you forgot to run:

npm install

Before to run gulp (when you are in the folder Babylon.js/Tools/Gulp).

Hope that helps.

Link to comment
Share on other sites

On 10/28/2018 at 10:05 AM, JohnK said:

@DylanD glad to hear that you are going to make a contribution. Even though I have had a couple of PRs accepted I still have problems when I do it usually with gulp. I think I will see if I can do a re-write that takes people who know little about git and github (as I was once and still no expert) on a step by step journey through it. I may be wrong but I think you are missing a repository on github and a fork of babylon.js in your repository. You can only issue a pull request from your fork to  that of babylon.js.

More here on forks

Having made and tested your local version you then need to check your version is built on the latest version of the babylon.js repository by doing a pull from that repository and then doing a push to your repository before issuing a pull request.

Pull and Pull Request difference

If you use git pull, you pull the changes from the remote repository into yours.

If you send a pull request to another repository, you ask their maintainers to pull your changes into theirs (you more or less ask them to use a git pull from your repository). From stackoverflow

 

Thank you!  I have been wanting to start contributing however, work and school have made it very difficult to keep up with everything.

 

After reading this i believe my problem is that i did not make a fork of the babylonjs master, when i clone it i assumed that was creating a fork, although i somewhat understand that, making a clone is not also creating a fork.  So i need to create a fork of babylon master then clone that, add my changes to that, then make make a pull request. Hopefully i will be able to try this soon!

 

I will keep this topic posted with my updates!

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Loading...
 Share

  • Recently Browsing   0 members

    • No registered users viewing this page.
×
×
  • Create New...