| Re: [udig-dev] Style Bug Fixes Pull Request |
Emily, Thanks for the fix!Like commented at the pull request itself its necessary to sign the commit. Since this fix is less than 1000 lines you only need to sign your commits with the option -s.git commit -s -m "message"The CI build is kind of broken because it does a full build (downloads dependencies, install environment (sphinx), build docs as a pre-build step and finally runs maven with sdk and product profiles). For details have a look at .travis.yml file in root checkout.IMHO we should deaktivate it since we have a hundson instance at locationtech (https://hudson.locationtech.org/udig/). We need to figure out how to setup CI-Builds for github pull requests..HTHFrank2015-03-03 19:00 GMT+01:00 Emily Gouge <egouge@xxxxxxxxxxxxxxx>:I made a few minor updates the styling UI to fix some bugs with the polygon themed styles. I tried to make a pull request:
https://github.com/locationtech/udig-platform/pull/191
The first problem with this is that I did not have a valid CLA. I think I fixed this.
The second problem seems to be with CI build. I don't really know what to do with this problem. Thoughts?
Thanks,
Emily
_______________________________________________
udig-dev mailing list
udig-dev@xxxxxxxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://locationtech.org/mailman/listinfo/udig-dev