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

feature: Implement artifacts deletion by gpocentek · Pull Request #767 · python-gitlab/python-gitlab · GitHub

Repository navigation

feature: Implement artifacts deletion - #767

Merged
max-wittig merged 1 commit into
masterfrom
fix/744/delete_artifacts
Jun 17, 2019
Merged

max-wittig merged 1 commit into
masterfrom
fix/744/delete_artifacts

Conversation

Copy link
Copy Markdown
Contributor

Closes #744

max-wittig self-assigned this May 9, 2019

max-wittig commented May 9, 2019 •
edited
Loading

Copy link
Copy Markdown
Member

A line is still to long for the CI pipeline to pass and would you mind rebasing and changing your commit message to match our conventional changelog?

max-wittig assigned gpocentek and unassigned max-wittig May 24, 2019

Copy link
Copy Markdown
Member

@gpocentek Would you mind, rebasing or should I take over?

gpocentek force-pushed the fix/744/delete_artifacts branch from 71010f6 to af45502 Compare June 8, 2019 06:00

Copy link
Copy Markdown
Contributor Author

Sorry for the delay...

gpocentek force-pushed the fix/744/delete_artifacts branch from af45502 to b148a2b Compare June 8, 2019 06:37
Comment thread gitlab/v4/objects.py Outdated
gpocentek force-pushed the fix/744/delete_artifacts branch from b148a2b to 07e01f8 Compare June 10, 2019 12:03

Copy link
Copy Markdown
Contributor Author

This requires #791

gpocentek force-pushed the fix/744/delete_artifacts branch from 07e01f8 to ab8c252 Compare June 10, 2019 12:18
gpocentek force-pushed the fix/744/delete_artifacts branch from ab8c252 to 76b6e1f Compare June 10, 2019 13:05

Copy link
Copy Markdown
Contributor Author

I can't reproduce the tests failure locally...

Copy link
Copy Markdown
Member

I will merge this now. Tests are passing locally.

max-wittig merged commit 4e1dd27 into master Jun 17, 2019
max-wittig deleted the fix/744/delete_artifacts branch June 17, 2019 15:56
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.

Add support for deleting job artifacts

2 participants


Back | FazBrowse Home | New Git URL