<div dir="auto">While I agree googling is an essential skill, we should probably add it in docs, no?</div><br><div class="gmail_quote"><div dir="ltr">On Wed, 13 Mar 2019, 2:20 am James Cameron, <<a href="mailto:quozl@laptop.org">quozl@laptop.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Here's how to do it;<br>
<br>
<a href="https://help.github.com/en/articles/creating-a-pull-request" rel="noreferrer noreferrer" target="_blank">https://help.github.com/en/articles/creating-a-pull-request</a><br>
<br>
Our Sugar Labs documents for contributing don't have every answer,<br>
sorry. Some skills are external.<br>
<br>
By the way, if you had made a local "git clone" on your own computer,<br>
then done a "git push", the server will have sent back instructions<br>
how to make a pull request.<br>
<br>
On Wed, Mar 13, 2019 at 01:54:51AM +0530, navyasri reddy wrote:<br>
> I have changed the readme and pushed to forked repo in my github account. How<br>
> do I raise a pull request to original repo. I have seen the docs for<br>
> contributing but didn't get it much.<br>
> <br>
> On Wed, Mar 13, 2019 at 12:15 AM Sumit Srivastava <[1]<a href="mailto:sumitsrisumit@gmail.com" target="_blank" rel="noreferrer">sumitsrisumit@gmail.com</a>><br>
> wrote:<br>
> <br>
> Yes, Jaskirat. The reply all button is a wonderful little thing and poses<br>
> much less friction than new members would otherwise experience.<br>
> <br>
> On Wed, 13 Mar 2019, 12:06 am Jaskirat Singh, <[2]<a href="mailto:juskirat2000@gmail.com" target="_blank" rel="noreferrer">juskirat2000@gmail.com</a>><br>
> wrote:<br>
> <br>
> Thanks Sumit, <br>
> <br>
> This sounds like a better choice for the people helping new<br>
> contributors to get started. <br>
> <br>
> On Wed, 13 Mar 2019, 12:04 am Sumit Srivastava, <[3]<br>
> <a href="mailto:sumitsrisumit@gmail.com" target="_blank" rel="noreferrer">sumitsrisumit@gmail.com</a>> wrote:<br>
> <br>
> Navya,<br>
> <br>
> Good to see you figuring out the codebase! Check out instructions<br>
> on downloading and running musicblocks. It should be present in the<br>
> musicblocks documentation. It states:<br>
> <br>
> "If you want to run Music Blocks offline, download this repo and<br>
> point your browser to the index.html file found in the musicblocks<br>
> directory on your local file system."<br>
> <br>
> Also, if you post your questions directly to [4]<br>
> <a href="mailto:sugar-devel@lists.sugarlabs.org" target="_blank" rel="noreferrer">sugar-devel@lists.sugarlabs.org</a> , more people can come in and<br>
> contribute. I am sorry if you had a rough start but rest assured,<br>
> the community is awesome! <br>
> <br>
> I'll be available at sugar-devel mailing list and so would be a lot<br>
> other experienced developers in the community. Always better to<br>
> expose your questions to more people, right? For this thread, you<br>
> can use "reply-all" button on gmail to directly post to sugarlabs<br>
> devel list and me.<br>
> <br>
> Keep asking questions! :)<br>
> <br>
> Regards<br>
> Sumit Srivastava<br>
> <br>
> On Tue, 12 Mar 2019, 11:51 pm navyasri reddy, <[5]<br>
> <a href="mailto:navyasrireddy1996@gmail.com" target="_blank" rel="noreferrer">navyasrireddy1996@gmail.com</a>> wrote:<br>
> <br>
> Hey,<br>
> Thanks for helping me out with this. I have found through the<br>
> other issues that the sound sample need to have same pitch<br>
> through out which is not mentioned in the readme and also<br>
> testing whether the sound is added properly or not is also not<br>
> there in the readme.<br>
> I am not sure how to test it. Can you direct me so that i can<br>
> test and update readme.<br>
> <br>
> Thanks.<br>
> <br>
> On Mon, Mar 11, 2019 at 7:08 PM Sumit Srivastava <[6]<br>
> <a href="mailto:sumitsrisumit@gmail.com" target="_blank" rel="noreferrer">sumitsrisumit@gmail.com</a>> wrote:<br>
> <br>
> Hi, navyasri! <br>
> <br>
> It'd be great if you post publicly to the sugar devel<br>
> mailing list. While I have no problem with replying to your<br>
> question privately, the community as a whole benefits more<br>
> from public discussions like these. <br>
> <br>
> Don't worry you can post all kinds of beginner questions,<br>
> even the one you think might be silly, to the mailing list.<br>
> :)<br>
> <br>
> Here are the steps to approach the issue:<br>
> <br>
> -Fork the branch.<br>
> <br>
> -Take a look at the instructions to add sample sound.<br>
> <br>
> -Try and do it yourself.<br>
> <br>
> -Did you find instructions were useful? Do they need to be<br>
> updated? (Hint: Yes, because of which the issue was<br>
> created)<br>
> <br>
> -Figure out how can you better explain the process you<br>
> followed to add a sample sound.<br>
> <br>
> -Write it down.<br>
> <br>
> -Commit to your branch, test, and then request to merge.<br>
> <br>
> Regards<br>
> Sumit Srivastava<br>
> <br>
> On Mon, 11 Mar 2019, 2:16 pm navyasri reddy, <[7]<br>
> <a href="mailto:navyasrireddy1996@gmail.com" target="_blank" rel="noreferrer">navyasrireddy1996@gmail.com</a>> wrote:<br>
> <br>
> Hey,<br>
> This is navyasri. I wan to contribute to music blocks<br>
> by starting with some basic contribution (below<br>
> attached). This is about updating readme but I am not<br>
> clear with the issue. There's no reply to extra<br>
> information on this issue. A help for this would be<br>
> appreciated.<br>
> [8]<a href="https://github.com/sugarlabs/musicblocks/issues/1468" rel="noreferrer noreferrer" target="_blank">https://github.com/sugarlabs/musicblocks/issues/1468</a><br>
> <br>
> Thanks, <br>
> navyasri<br>
> <br>
> _______________________________________________<br>
> Sugar-devel mailing list<br>
> [9]<a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank" rel="noreferrer">Sugar-devel@lists.sugarlabs.org</a><br>
> [10]<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" rel="noreferrer noreferrer" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
> <br>
> References:<br>
> <br>
> [1] mailto:<a href="mailto:sumitsrisumit@gmail.com" target="_blank" rel="noreferrer">sumitsrisumit@gmail.com</a><br>
> [2] mailto:<a href="mailto:juskirat2000@gmail.com" target="_blank" rel="noreferrer">juskirat2000@gmail.com</a><br>
> [3] mailto:<a href="mailto:sumitsrisumit@gmail.com" target="_blank" rel="noreferrer">sumitsrisumit@gmail.com</a><br>
> [4] mailto:<a href="mailto:sugar-devel@lists.sugarlabs.org" target="_blank" rel="noreferrer">sugar-devel@lists.sugarlabs.org</a><br>
> [5] mailto:<a href="mailto:navyasrireddy1996@gmail.com" target="_blank" rel="noreferrer">navyasrireddy1996@gmail.com</a><br>
> [6] mailto:<a href="mailto:sumitsrisumit@gmail.com" target="_blank" rel="noreferrer">sumitsrisumit@gmail.com</a><br>
> [7] mailto:<a href="mailto:navyasrireddy1996@gmail.com" target="_blank" rel="noreferrer">navyasrireddy1996@gmail.com</a><br>
> [8] <a href="https://github.com/sugarlabs/musicblocks/issues/1468" rel="noreferrer noreferrer" target="_blank">https://github.com/sugarlabs/musicblocks/issues/1468</a><br>
> [9] mailto:<a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank" rel="noreferrer">Sugar-devel@lists.sugarlabs.org</a><br>
> [10] <a href="http://lists.sugarlabs.org/listinfo/sugar-devel" rel="noreferrer noreferrer" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
<br>
> _______________________________________________<br>
> Sugar-devel mailing list<br>
> <a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank" rel="noreferrer">Sugar-devel@lists.sugarlabs.org</a><br>
> <a href="http://lists.sugarlabs.org/listinfo/sugar-devel" rel="noreferrer noreferrer" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
<br>
<br>
-- <br>
James Cameron<br>
<a href="http://quozl.netrek.org/" rel="noreferrer noreferrer" target="_blank">http://quozl.netrek.org/</a><br>
</blockquote></div>