Skip to content

models/webhook.go linting#198

Merged
lunny merged 1 commit into
go-gitea:masterfrom
strk:webhook-lint
Nov 22, 2016
Merged

models/webhook.go linting#198
lunny merged 1 commit into
go-gitea:masterfrom
strk:webhook-lint

Conversation

@strk
Copy link
Copy Markdown
Member

@strk strk commented Nov 19, 2016

still incomplete

@codecov-io
Copy link
Copy Markdown

codecov-io commented Nov 19, 2016 •
edited
Loading

Current coverage is 3.03% (diff: 0.00%)

Merging #198 into master will not change coverage

@@            master      #198   diff @@
========================================
  Files           33        33          
  Lines         8106      8106          
  Methods          0         0          
  Messages         0         0          
  Branches         0         0          
========================================
  Hits           246       246          
  Misses        7840      7840          
  Partials        20        20

Powered by Codecov. Last update b2cce12...6cf115b

@andreynering andreynering added the pr/wip This PR is not ready for review label Nov 19, 2016
@strk strk changed the title [WIP] Some linting for webhook models/webhook.go linting Nov 20, 2016
@strk
Copy link
Copy Markdown
Member Author

strk commented Nov 20, 2016

This is ready for merge, please review.

@andreynering
Copy link
Copy Markdown
Contributor

andreynering commented Nov 20, 2016

This is not building:

# code.gitea.io/gitea/models
models/webhook.go:395: t.SimpleMarshalJSON undefined (type *HookTask has no field or method SimpleMarshalJSON, but does have simpleMarshalJSON)
models/webhook.go:398: t.SimpleMarshalJSON undefined (type *HookTask has no field or method SimpleMarshalJSON, but does have simpleMarshalJSON)

@lunny
Copy link
Copy Markdown
Member

lunny commented Nov 21, 2016

^

@lunny lunny added this to the 1.0.0 milestone Nov 21, 2016
@strk
Copy link
Copy Markdown
Member Author

strk commented Nov 21, 2016

Build should be fixed now, sorry for the noise

@lunny
Copy link
Copy Markdown
Member

lunny commented Nov 21, 2016

LGTM

@lunny lunny added type/docs This PR mainly updates/creates documentation type/enhancement An improvement of existing functionality and removed pr/wip This PR is not ready for review labels Nov 21, 2016
@thibaultmeyer
Copy link
Copy Markdown
Contributor

thibaultmeyer commented Nov 21, 2016

LGTM

@lunny lunny merged commit c25063d into go-gitea:master Nov 22, 2016
@strk strk deleted the webhook-lint branch November 22, 2016 08:27
@lunny
Copy link
Copy Markdown
Member

lunny commented Nov 28, 2016

resolved #70

@tboerger tboerger added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. topic/code-linting and removed type/docs This PR mainly updates/creates documentation type/enhancement An improvement of existing functionality labels Nov 28, 2016
@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

No reviews

Assignees

No one assigned

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. topic/code-linting

Projects

None yet

Milestone

1.0.0

Development

Successfully merging this pull request may close these issues.

6 participants

@strk @codecov-io @andreynering @lunny @thibaultmeyer @tboerger