Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removed source builds from recent commit f3c15d0 #13186

Merged
merged 3 commits into from
Nov 11, 2016
Merged

Removed source builds from recent commit f3c15d0 #13186

merged 3 commits into from
Nov 11, 2016

Conversation

kmhallen
Copy link
Contributor

No description provided.

@wjwwood
Copy link
Member

wjwwood commented Nov 11, 2016

We can certainly do this, but if the reason is to disable the build farm trying to build each commit you make, you can do that with the test_commits: false line inside the source: block.

@dirk-thomas
Copy link
Member

The advantage of the approach @wjwwood mentioned is that users can still easily fetch your code using tools like rosinstall_generator.

@wjwwood
Copy link
Member

wjwwood commented Nov 11, 2016

Hmm, looks like something went wrong and now you're partially removing the doc sections. You'll either have to undo that or remove the doc: line as well (if you're intention is disable doc jobs). You want something like this:

  dataspeed_can:
    doc:
      type: hg
      url: https://bitbucket.org/dataspeedinc/dataspeed_can
      version: default
    source:
      test_commits: false
      type: hg
      url: https://bitbucket.org/dataspeedinc/dataspeed_can
      version: default
    status: developed

@kmhallen
Copy link
Contributor Author

Fixed, silly mistake...

@wjwwood
Copy link
Member

wjwwood commented Nov 11, 2016

No worries, thanks for fixing it up quickly.

@wjwwood wjwwood merged commit 8afb7b6 into ros:master Nov 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants