Skip to content

Fix "Error: No issue number specified" when pushing #1393

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
Merged
merged 2 commits into from
Apr 1, 2017
Merged

Fix "Error: No issue number specified" when pushing #1393

merged 2 commits into from
Apr 1, 2017

Conversation

@agrn
Copy link
Contributor

@agrn agrn commented Mar 27, 2017

This patch fixes issue #1111

@andreynering
Copy link
Contributor

andreynering commented Mar 27, 2017

LGTM

@andreynering andreynering added this to the 1.2.0 milestone Mar 27, 2017
@tboerger tboerger added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Mar 27, 2017
@lunny
Copy link
Member

lunny commented Mar 28, 2017

LGTM

@tboerger tboerger removed the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Mar 28, 2017
@lunny
Copy link
Member

lunny commented Mar 28, 2017

let L-G-T-M work

@appleboy
Copy link
Member

appleboy commented Mar 28, 2017

LGTM

@tboerger tboerger added the lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. label Mar 28, 2017
@lunny
Copy link
Member

lunny commented Mar 31, 2017

@agrn could make an empty force push to this PR to let drone work?

@agrn
Copy link
Contributor Author

agrn commented Mar 31, 2017

OK, did it

@lunny lunny merged commit 88112a5 into go-gitea:master Apr 1, 2017
@mjwwit mjwwit mentioned this pull request Apr 3, 2017
Closed
@lunny
Copy link
Member

lunny commented May 19, 2017

@agrn could you backport this to branch release/v1.1 ?

@agrn agrn mentioned this pull request May 19, 2017
Merged
@agrn
Copy link
Contributor Author

agrn commented May 19, 2017

I included #1220 in the PR as it's also part of the bugfix.

@lunny lunny added the backport/done All backports for this PR have been created label May 19, 2017
lunny pushed a commit that referenced this pull request May 20, 2017
@agrn @lunny
Backport bugfixes #1220 and #1393 to v1.1 (#1758)
4fd55d8 
* Fix "Error: No issue number specified"  when pushing

* fix wrong log when push to a new branch (#1220)

* Removing empty file modules/github/issues.go
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@appleboy appleboy appleboy approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

backport/done All backports for this PR have been created lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug

Projects

None yet

Milestone

1.2.0

Development

Successfully merging this pull request may close these issues.

5 participants

@agrn @andreynering @lunny @appleboy @tboerger