Skip to content

add commit file statuses #5607

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
wants to merge 9 commits into from

Conversation

lunny
Copy link
Member

@lunny lunny commented Dec 29, 2018

Should fix #4313 and blocked by go-gitea/go-sdk#135 and go-gitea/git#137

@lafriks lafriks added the type/enhancement An improvement of existing functionality label Jan 2, 2019
@lafriks lafriks added this to the 1.8.0 milestone Jan 2, 2019
@lafriks
Copy link
Member

lafriks commented Jan 2, 2019

Dependencies has been merged

@bkcsoft bkcsoft added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jan 2, 2019
@lunny lunny force-pushed the lunny/add_commit_file_status branch from ff91fb6 to 2c2b00a Compare January 3, 2019 09:13
@lunny
Copy link
Member Author

lunny commented Jan 3, 2019

@lafriks updated

@lunny lunny force-pushed the lunny/add_commit_file_status branch from 2c2b00a to 74e919e Compare January 3, 2019 16:25
@lafriks
Copy link
Member

lafriks commented Jan 5, 2019

Looks like swagger needs updating

@lunny lunny force-pushed the lunny/add_commit_file_status branch from 74e919e to 4b28826 Compare January 14, 2019 07:17
@lafriks
Copy link
Member

lafriks commented Jan 15, 2019

Do not add newline in the end of v1_json.tmpl otherwise drone will fail

@lunny lunny force-pushed the lunny/add_commit_file_status branch from 5201f89 to f7c5a7a Compare January 16, 2019 03:33
@lunny
Copy link
Member Author

lunny commented Jan 16, 2019

@lafriks done.

@lunny lunny force-pushed the lunny/add_commit_file_status branch 2 times, most recently from e0ec323 to cdbb703 Compare January 19, 2019 16:48
@lunny lunny force-pushed the lunny/add_commit_file_status branch from cdbb703 to 9ea8a32 Compare February 7, 2019 07:51
@techknowlogick techknowlogick modified the milestones: 1.8.0, 1.9.0 Feb 19, 2019
@lunny lunny force-pushed the lunny/add_commit_file_status branch from 9ea8a32 to 41ec892 Compare April 2, 2019 02:16
@decke
Copy link
Contributor

decke commented May 23, 2019

Is there any work needed before it can get merged?

@@ -60,6 +60,8 @@ Values containing `#` or `;` must be quoted using `` ` `` or `"""`.
the top of the list. Name must match file name in conf/license or custom/conf/license.
- `DISABLE_HTTP_GIT`: **false**: Disable the ability to interact with repositories over the
HTTP protocol.
- `ACCESS_CONTROL_ALLOW_ORIGIN`: ****: Value for Access-Control-Allow-Origin header, default is not to present.
Copy link
Member

Choose a reason for hiding this comment

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

I think this is for a different PR?

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- `ACCESS_CONTROL_ALLOW_ORIGIN`: ****: Value for Access-Control-Allow-Origin header, default is not to present.

@@ -60,6 +60,8 @@ Values containing `#` or `;` must be quoted using `` ` `` or `"""`.
the top of the list. Name must match file name in conf/license or custom/conf/license.
- `DISABLE_HTTP_GIT`: **false**: Disable the ability to interact with repositories over the
HTTP protocol.
- `ACCESS_CONTROL_ALLOW_ORIGIN`: ****: Value for Access-Control-Allow-Origin header, default is not to present.
WARNING: This maybe harmful to you website if you do not give it a right value.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
WARNING: This maybe harmful to you website if you do not give it a right value.

@@ -32,6 +32,8 @@ menu:
- `DEFAULT_PRIVATE`: 默认创建的git工程为私有。 可以是`last`, `private` 或 `public`。默认值是 `last`表示用户最后创建的Repo的选择。
- `MAX_CREATION_LIMIT`: 全局最大每个用户创建的git工程数目, `-1` 表示没限制。
- `PULL_REQUEST_QUEUE_LENGTH`: 小心:合并请求测试队列的长度,尽量放大。
- `ACCESS_CONTROL_ALLOW_ORIGIN`: ****: Access-Control-Allow-Origin 头的值,默认为空。
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- `ACCESS_CONTROL_ALLOW_ORIGIN`: ****: Access-Control-Allow-Origin 头的值,默认为空。

@@ -32,6 +32,8 @@ menu:
- `DEFAULT_PRIVATE`: 默认创建的git工程为私有。 可以是`last`, `private` 或 `public`。默认值是 `last`表示用户最后创建的Repo的选择。
- `MAX_CREATION_LIMIT`: 全局最大每个用户创建的git工程数目, `-1` 表示没限制。
- `PULL_REQUEST_QUEUE_LENGTH`: 小心:合并请求测试队列的长度,尽量放大。
- `ACCESS_CONTROL_ALLOW_ORIGIN`: ****: Access-Control-Allow-Origin 头的值,默认为空。
警告:如果您不熟悉这方面的设置请不要修改此值,可能影响网站安全。
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
警告:如果您不熟悉这方面的设置请不要修改此值,可能影响网站安全。

@lunny lunny force-pushed the lunny/add_commit_file_status branch from a2a1324 to bf4bcfb Compare May 24, 2019 04:41
@techknowlogick techknowlogick modified the milestones: 1.9.0, 1.10.0 Jun 4, 2019
@lunny lunny mentioned this pull request Jul 30, 2019
@lunny lunny closed this Jul 30, 2019
@lunny lunny deleted the lunny/add_commit_file_status branch July 30, 2019 05:54
@lunny lunny removed this from the 1.10.0 milestone Jul 30, 2019
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. type/enhancement An improvement of existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Webhook POST data doesn't pass added/modified/removed info in commits section
6 participants