Skip to content
Snippets Groups Projects

Grapify builds API

Merged username-removed-2900 requested to merge grapify-builds-api into master

What does this MR do?

Add the Grape-DSL to the builds API.

What are the relevant issue numbers?

Related to #22928 (moved)

The artifacts API directly downloads a file rather then returning a JSON entity.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • username-removed-2900 Resolved all discussions

    Resolved all discussions

  • Added 1 commit:

    Compare with previous version

  • @rymai Thanks for the review. I updated the MR.

  • Milestone changed to %8.14

  • Reassigned to @razer6

  • The build is failing because scope can either be a string or a hash in the from scope"=>{"0"=>"pending", "1"=>"running"}, which is not yet covered by the DSL specification.

  • @razer6 Why do we accept "scope"=>{"0"=>"pending", "1"=>"running"}?

  • @rymai I found that in the test here. The idea is to filter in different scopes. The syntax of this test is translated to the hash above.

  • @razer6 Right, and it's documented!

  • Added 1 commit:

    Compare with previous version

  • Reassigned to @rymai

  • @rymai I fixed that issue to support multiple types.

  • username-removed-2900 Resolved all discussions

    Resolved all discussions

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading