Add Hook #1644

Merged
lunny merged 2 commits from yxlimo/xorm:master into master 2020-04-09 06:03:50 +00:00
Contributor
No description provided.
Owner

@limo.creed Maybe we should move the Hook to a standalone package? And could you add some tests?

@limo.creed Maybe we should move the Hook to a standalone package? And could you add some tests?
lunny added the
kind
feature
label 2020-04-02 09:33:24 +00:00
Author
Contributor

@limo.creed Maybe we should move the Hook to a standalone package? And could you add some tests?

i will add some tests later.

and i am consider that use LogContext is not a good way.

maybe use a new struct instead is better, like QueryContext ?

> @limo.creed Maybe we should move the Hook to a standalone package? And could you add some tests? i will add some tests later. and i am consider that use `LogContext` is not a good way. maybe use a new struct instead is better, like `QueryContext` ?
Author
Contributor

@lunny i added some tests and move it alone

@lunny i added some tests and move it alone
Owner

Please resolve the conflicts.

Please resolve the conflicts.
Author
Contributor

@lunny resolved

@lunny resolved
Owner

How about move Hook to contexts package but not a new sub package? This should be the last thinking about this PR.

How about move `Hook` to `contexts` package but not a new sub package? This should be the last thinking about this PR.
Author
Contributor

seems good. i have moved in contexts

seems good. i have moved in `contexts`
lunny approved these changes 2020-04-09 06:03:30 +00:00
Dismissed
lunny closed this pull request 2020-04-09 06:03:50 +00:00
lunny added this to the 1.0.2 milestone 2020-04-13 13:21:03 +00:00
Sign in to join this conversation.
No description provided.