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.

Sign up for GitHub

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

Jump to bottom

Add integration test for repository forking #1896

Merged
merged 1 commit into from
Jun 7, 2017
Merged

Add integration test for repository forking #1896

merged 1 commit into from
Jun 7, 2017

Conversation

typeless
Copy link
Contributor

@typeless typeless commented Jun 7, 2017

Add integration test for repository forking.

@typeless typeless changed the title Add integration test for repository forking [WIP] Add integration test for repository forking Jun 7, 2017
@strk
Copy link
Member

strk commented Jun 7, 2017

LGTM go tests go !

@tboerger tboerger added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Jun 7, 2017
@lafriks
Copy link
Member

lafriks commented Jun 7, 2017

It would be great if after POST request it would check also forked repository GET request that is redirected to from POST request response

@typeless
Copy link
Contributor Author

typeless commented Jun 7, 2017

@lafriks Good idea.

@typeless
Copy link
Contributor Author

typeless commented Jun 7, 2017 •
edited
Loading

Please hold on, I am working on some tests for pulls-merging which will be combined into this PR.
I changed my mind. I will base my follow-up PRs on this.

@typeless typeless force-pushed the add-integration-test-for-forking branch from 75aee1c to 30a8316 Compare June 7, 2017 12:02
@lafriks
Copy link
Member

lafriks commented Jun 7, 2017

LGTM

@tboerger tboerger added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Jun 7, 2017
@typeless typeless changed the title [WIP] Add integration test for repository forking Add integration test for repository forking Jun 7, 2017
@andreynering
Copy link
Contributor

Thank you very much

@andreynering andreynering merged commit 815715e into go-gitea:master Jun 7, 2017
@lunny lunny added this to the 1.2.0 milestone Jun 8, 2017
@typeless typeless deleted the add-integration-test-for-forking branch April 3, 2019 05:00
@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.
Reviewers
No reviews
Assignees
No one assigned
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/testing
Projects
None yet
Milestone
1.2.0
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@typeless @strk @lafriks @andreynering @lunny @tboerger