<div dir="auto"><div dir="auto">James, that's great! Yup, I agree. ;-)<br></div><br><div class="gmail_quote" dir="auto"><div dir="ltr">On Wed, 13 Mar 2019, 4:18 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">We have a link in contributing.md;<br>
<br>
"We use the pull-request model, see [GitHub's help on pull-request](<a href="https://help.github.com/articles/using-pull-requests)" rel="noreferrer noreferrer" target="_blank">https://help.github.com/articles/using-pull-requests)</a>."<br>
<br>
But people can't be expected to remember everything.  ;-)<br>
<br>
On Wed, Mar 13, 2019 at 02:24:44AM +0530, Sumit Srivastava wrote:<br>
> While I agree googling is an essential skill, we should probably add it in<br>
> docs, no?<br>
> <br>
> On Wed, 13 Mar 2019, 2:20 am James Cameron, <[1]<a href="mailto:quozl@laptop.org" target="_blank" rel="noreferrer">quozl@laptop.org</a>> wrote:<br>
> <br>
>     Here's how to do it;<br>
> <br>
>     [2]<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.<br>
>     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][3]<br>
>     <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<br>
>     poses<br>
>     >     much less friction than new members would otherwise experience.<br>
>     ><br>
>     >     On Wed, 13 Mar 2019, 12:06 am Jaskirat Singh, <[2][4]<br>
>     <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>
>     >         [5]<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<br>
>     instructions<br>
>     >             on downloading and running musicblocks. It should be present<br>
>     in the<br>
>     >             musicblocks documentation. It states:<br>
>     ><br>
>     >             "If you want to run Music Blocks offline, download this repo<br>
>     and<br>
>     >             point your browser to the index.html file found in the<br>
>     musicblocks<br>
>     >             directory on your local file system."<br>
>     ><br>
>     >             Also, if you post your questions directly to [4]<br>
>     >             [6]<a href="mailto:sugar-devel@lists.sugarlabs.org" target="_blank" rel="noreferrer">sugar-devel@lists.sugarlabs.org</a> , more people can come in<br>
>     and<br>
>     >             contribute. I am sorry if you had a rough start but rest<br>
>     assured,<br>
>     >             the community is awesome! <br>
>     ><br>
>     >             I'll be available at sugar-devel mailing list and so would be<br>
>     a lot<br>
>     >             other experienced developers in the community. Always better<br>
>     to<br>
>     >             expose your questions to more people, right? For this thread,<br>
>     you<br>
>     >             can use "reply-all" button on gmail to directly post to<br>
>     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>
>     >             [7]<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<br>
>     the<br>
>     >                 other issues that the sound sample need to have same<br>
>     pitch<br>
>     >                 through out which is not mentioned in the readme and also<br>
>     >                 testing whether the sound is added properly or not is<br>
>     also not<br>
>     >                 there in the readme.<br>
>     >                 I am not sure how to test it. Can you direct me so that i<br>
>     can<br>
>     >                 test and update readme.<br>
>     ><br>
>     >                 Thanks.<br>
>     ><br>
>     >                 On Mon, Mar 11, 2019 at 7:08 PM Sumit Srivastava <[6]<br>
>     >                 [8]<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<br>
>     to your<br>
>     >                     question privately, the community as a whole benefits<br>
>     more<br>
>     >                     from public discussions like these. <br>
>     ><br>
>     >                     Don't worry you can post all kinds of beginner<br>
>     questions,<br>
>     >                     even the one you think might be silly, to the mailing<br>
>     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<br>
>     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<br>
>     you<br>
>     >                     followed to add a sample sound.<br>
>     ><br>
>     >                     -Write it down.<br>
>     ><br>
>     >                     -Commit to your branch, test, and then request to<br>
>     merge.<br>
>     ><br>
>     >                     Regards<br>
>     >                     Sumit Srivastava<br>
>     ><br>
>     >                     On Mon, 11 Mar 2019, 2:16 pm navyasri reddy, <[7]<br>
>     >                     [9]<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<br>
>     blocks<br>
>     >                         by starting with some basic contribution (below<br>
>     >                         attached). This is about updating readme but I am<br>
>     not<br>
>     >                         clear with the issue. There's no reply to extra<br>
>     >                         information on this issue. A help for this would<br>
>     be<br>
>     >                         appreciated.<br>
>     >                         [8][10]<a href="https://github.com/sugarlabs/musicblocks/" rel="noreferrer noreferrer" target="_blank">https://github.com/sugarlabs/musicblocks/</a><br>
>     issues/1468<br>
>     ><br>
>     >                         Thanks, <br>
>     >                         navyasri<br>
>     ><br>
>     >             _______________________________________________<br>
>     >             Sugar-devel mailing list<br>
>     >             [9][11]<a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank" rel="noreferrer">Sugar-devel@lists.sugarlabs.org</a><br>
>     >             [10][12]<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:[13]<a href="mailto:sumitsrisumit@gmail.com" target="_blank" rel="noreferrer">sumitsrisumit@gmail.com</a><br>
>     > [2] mailto:[14]<a href="mailto:juskirat2000@gmail.com" target="_blank" rel="noreferrer">juskirat2000@gmail.com</a><br>
>     > [3] mailto:[15]<a href="mailto:sumitsrisumit@gmail.com" target="_blank" rel="noreferrer">sumitsrisumit@gmail.com</a><br>
>     > [4] mailto:[16]<a href="mailto:sugar-devel@lists.sugarlabs.org" target="_blank" rel="noreferrer">sugar-devel@lists.sugarlabs.org</a><br>
>     > [5] mailto:[17]<a href="mailto:navyasrireddy1996@gmail.com" target="_blank" rel="noreferrer">navyasrireddy1996@gmail.com</a><br>
>     > [6] mailto:[18]<a href="mailto:sumitsrisumit@gmail.com" target="_blank" rel="noreferrer">sumitsrisumit@gmail.com</a><br>
>     > [7] mailto:[19]<a href="mailto:navyasrireddy1996@gmail.com" target="_blank" rel="noreferrer">navyasrireddy1996@gmail.com</a><br>
>     > [8] [20]<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:[21]<a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank" rel="noreferrer">Sugar-devel@lists.sugarlabs.org</a><br>
>     > [10] [22]<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>
>     > [23]<a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank" rel="noreferrer">Sugar-devel@lists.sugarlabs.org</a><br>
>     > [24]<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>
>     James Cameron<br>
>     [25]<a href="http://quozl.netrek.org/" rel="noreferrer noreferrer" target="_blank">http://quozl.netrek.org/</a><br>
> <br>
> References:<br>
> <br>
> [1] mailto:<a href="mailto:quozl@laptop.org" target="_blank" rel="noreferrer">quozl@laptop.org</a><br>
> [2] <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>
> [3] mailto:<a href="mailto:sumitsrisumit@gmail.com" target="_blank" rel="noreferrer">sumitsrisumit@gmail.com</a><br>
> [4] mailto:<a href="mailto:juskirat2000@gmail.com" target="_blank" rel="noreferrer">juskirat2000@gmail.com</a><br>
> [5] mailto:<a href="mailto:sumitsrisumit@gmail.com" target="_blank" rel="noreferrer">sumitsrisumit@gmail.com</a><br>
> [6] mailto:<a href="mailto:sugar-devel@lists.sugarlabs.org" target="_blank" rel="noreferrer">sugar-devel@lists.sugarlabs.org</a><br>
> [7] mailto:<a href="mailto:navyasrireddy1996@gmail.com" target="_blank" rel="noreferrer">navyasrireddy1996@gmail.com</a><br>
> [8] mailto:<a href="mailto:sumitsrisumit@gmail.com" target="_blank" rel="noreferrer">sumitsrisumit@gmail.com</a><br>
> [9] mailto:<a href="mailto:navyasrireddy1996@gmail.com" target="_blank" rel="noreferrer">navyasrireddy1996@gmail.com</a><br>
> [10] <a href="https://github.com/sugarlabs/musicblocks/issues/1468" rel="noreferrer noreferrer" target="_blank">https://github.com/sugarlabs/musicblocks/issues/1468</a><br>
> [11] mailto:<a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank" rel="noreferrer">Sugar-devel@lists.sugarlabs.org</a><br>
> [12] <a href="http://lists.sugarlabs.org/listinfo/sugar-devel" rel="noreferrer noreferrer" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
> [13] mailto:<a href="mailto:sumitsrisumit@gmail.com" target="_blank" rel="noreferrer">sumitsrisumit@gmail.com</a><br>
> [14] mailto:<a href="mailto:juskirat2000@gmail.com" target="_blank" rel="noreferrer">juskirat2000@gmail.com</a><br>
> [15] mailto:<a href="mailto:sumitsrisumit@gmail.com" target="_blank" rel="noreferrer">sumitsrisumit@gmail.com</a><br>
> [16] mailto:<a href="mailto:sugar-devel@lists.sugarlabs.org" target="_blank" rel="noreferrer">sugar-devel@lists.sugarlabs.org</a><br>
> [17] mailto:<a href="mailto:navyasrireddy1996@gmail.com" target="_blank" rel="noreferrer">navyasrireddy1996@gmail.com</a><br>
> [18] mailto:<a href="mailto:sumitsrisumit@gmail.com" target="_blank" rel="noreferrer">sumitsrisumit@gmail.com</a><br>
> [19] mailto:<a href="mailto:navyasrireddy1996@gmail.com" target="_blank" rel="noreferrer">navyasrireddy1996@gmail.com</a><br>
> [20] <a href="https://github.com/sugarlabs/musicblocks/issues/1468" rel="noreferrer noreferrer" target="_blank">https://github.com/sugarlabs/musicblocks/issues/1468</a><br>
> [21] mailto:<a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank" rel="noreferrer">Sugar-devel@lists.sugarlabs.org</a><br>
> [22] <a href="http://lists.sugarlabs.org/listinfo/sugar-devel" rel="noreferrer noreferrer" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
> [23] mailto:<a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank" rel="noreferrer">Sugar-devel@lists.sugarlabs.org</a><br>
> [24] <a href="http://lists.sugarlabs.org/listinfo/sugar-devel" rel="noreferrer noreferrer" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
> [25] <a href="http://quozl.netrek.org/" rel="noreferrer noreferrer" target="_blank">http://quozl.netrek.org/</a><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></div>