[Scons-users] SWIG emitter patch

[Bill Deegan]

Evgenii,

Take a look at this page:
https://github.com/SCons/scons/wiki/GitWorkflows

It has some basic instructions on how to submit a patch as a pull requests.

And:
https://github.com/SCons/scons/wiki/TestingMethodology
On testing.
Note, that you can likely look at the tests in : test/SWIG/ for some
examples.

Please don't hesitate to ask for help here or in our IRC channel #scons on
freenode.net
(there's a web base chat at :http://webchat.freenode.net/ )

Thanks,
Bill
SCons Project Co-manager.

On Sun, Jan 21, 2018 at 11:32 AM, Mats Wichmann <mats at wichmann.us> wrote:

> On 01/21/2018 12:46 AM, E.Z. Evgenii wrote:
> > I can file a bug report, with this code piece, if that's what you mean.
> >
> > I am new here and don't know what pull request is.
> >
> > I also haven't tried the test set up. I suppose I could write a test for
> this, but it would take time to learn, and I don't have much spare time.
> >
> > I should write a unit test for my patch anyway...
> >
> > Evgenii
>
> a pull request means you leave a request on the github project page for
> your change to be merged (from a branch somewhere where it is already
> integrated).  explaining the full mechanism is beyond a message here.
> when you submit a pull requests, the tests will be automatically run, so
> this helps the maintainers evaluate the correctness of the change/test
> change.
>
>
> _______________________________________________
> Scons-users mailing list
> Scons-users at scons.org
> https://pairlist4.pair.net/mailman/listinfo/scons-users
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://pairlist4.pair.net/pipermail/scons-users/attachments/20180121/72ec60b0/attachment-0001.html>