FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Implement create pipeline API by mondwan · Pull Request #237 · python-gitlab/python-gitlab · GitHub

Repository navigation

Implement create pipeline API - #237

Merged
gpocentek merged 2 commits into
python-gitlab:masterfrom
mondwan:master
Mar 18, 2017
Merged

gpocentek merged 2 commits into
python-gitlab:masterfrom
mondwan:master

Conversation

mondwan commented Mar 17, 2017

Copy link
Copy Markdown
Contributor
  • Integrate with BaseManager
  • Add doc about how to create a pipeline

Reference:

* Integrate with BaseManager
* Add doc about how to create a pipeline

Reference:
* https://docs.gitlab.com/ee/api/pipelines.html#create-a-new-pipeline

mondwan commented Mar 17, 2017 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Notes: I am not able to run tox with py27 and py34 by default.

gpocentek left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Apart from a nitpick in the docs the patch looks good. If you could just do the minor update, I'll merge your change.

Thank you!

Comment thread docs/gl_objects/projects.py Outdated
# pipeline create
pipeline = gl.project_pipelines.create({'project_id': 1, 'ref': 'master'})
# or
project.pipelines.create({'ref': 'master'})

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Could you just do a minor edit for consistency:

pipeline = project.pipelines.create({'ref': 'master'})

mondwan commented Mar 17, 2017

Copy link
Copy Markdown
Contributor Author

Sure. I have updated the document as requested

gpocentek merged commit 8c27e70 into python-gitlab:master Mar 18, 2017

Copy link
Copy Markdown
Contributor

Thank you!

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL