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

Better CI Integration #996

Closed
4 tasks done
MorphBonehunter opened this issue Feb 21, 2017 · 13 comments
Closed
4 tasks done

Better CI Integration #996

MorphBonehunter opened this issue Feb 21, 2017 · 13 comments
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented type/feature Completely new functionality. Can only be merged if feature freeze is not active.
Milestone

Comments

@MorphBonehunter
Copy link
Contributor

MorphBonehunter commented Feb 21, 2017

Description

I'm using drone as CI System at the moment (test run) and actually i can not see if an repository has CI enabled (besides i look into the repo settings webhooks) or the build status.
Only posibility is to include the drone repo badget into the readme.md.

So it would be nice if the build status could be implemented into:

@MorphBonehunter
Copy link
Contributor Author

BTW, i'm not sure if this require also work on the drone project, but maybe if there is an plan how this could be implemented and the requirements are know, i could also open an request on the drone issue tracker.

@lunny lunny added this to the 1.x.x milestone Feb 21, 2017
@lunny lunny added the type/feature Completely new functionality. Can only be merged if feature freeze is not active. label Feb 21, 2017
@BlackVoid
Copy link

Duplicate of #357?

@lunny
Copy link
Member

lunny commented Feb 21, 2017

It's not the same thing. One is for API, one is on the UI before the commit.

@BlackVoid
Copy link

Fair enough

@tboerger
Copy link
Member

I'm sure we already got an issue for that.

@MorphBonehunter
Copy link
Contributor Author

Couldn't find something similar in the open issues.

@bkcsoft
Copy link
Member

bkcsoft commented Feb 22, 2017

All of those needs #357 though, this is an extension on that 🙂

@strk
Copy link
Member

strk commented Jan 7, 2018

Status API (#357) is reported as being merged by now, so what are we missing to show status in UI ?
And to have Drone update statuses ?

@lafriks
Copy link
Member

lafriks commented Jan 7, 2018

@strk drone does already update status. What we miss is #2519

@iddm
Copy link
Contributor

iddm commented Feb 23, 2018

I'd also love the status being shown right on the pull request's page as in github. Also, it would be nice to prevent merging of commits if the CI have not build it successfully yet.


@vijfhoek
Copy link
Contributor

vijfhoek commented Jan 4, 2019

For what it's worth, I made a userscript to bridge the gap until #2519 is merged. It currently only shows a status on the PR page itself, but it'll do:

image

@stale
Copy link

stale bot commented Mar 5, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

@stale stale bot added the issue/stale label Mar 5, 2019
@NotAFile
Copy link

NotAFile commented Mar 5, 2019

this issue is still relevant

@lunny lunny modified the milestones: 1.x.x, 1.10.0 Jul 16, 2019
@lunny lunny closed this as completed Sep 18, 2019
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented type/feature Completely new functionality. Can only be merged if feature freeze is not active.
Projects
None yet
Development

No branches or pull requests

10 participants