CI: Travis script fails for MR based on repos not in the schleuder/ namespace; how to test code in these MR
- If people open a MR which is based on a repo / branch which is not within the schleuder/ namespace, this code doesn't get mirrored to GitHub, therefore Travis doesn't run, which currently leads to the script failing, as the commit SHA isn't found, preventing merging the MR via the web interface.
- Besides this, (but this might be a different issue): The new code doesn't get tested, so we've got no clue, whether it's fine or not, apart from looking at it.
- Regarding the first problem: If the commit SHA isn't known to Travis, the script could just exit (
0
? 1
?) and accordingly print the reason.
- Regarding the second problem: We could ask the people who want to contribute to tell us in advance, give them developer access and allow them to push to branches below
contrib/*
, for example, which would allow them to push a branch like contrib/new-feature
, for example.