Skip to content

Remove Change.handleError - #2308

Merged
0candy merged 1 commit into
masterfrom
remove_change_handleError
May 9, 2016
Merged

Remove Change.handleError#2308
0candy merged 1 commit into
masterfrom
remove_change_handleError

Conversation

@0candy

Copy link
Copy Markdown
Contributor

@0candy
0candyforce-pushed the remove_change_handleError branch 2 times, most recently from 3f5d146 to 31390f6CompareMay 6, 2016 17:34
Comment thread3.0-RELEASE-NOTES.md Outdated
Please see [related code change](https://github.com/strongloop/loopback/pull/2174) here. No newline at end of file
Please see [related code change](https://github.com/strongloop/loopback/pull/2174) here.

## removing `Change.handleError`

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please use imperative as we already use elsewhere, i.e.

## remove `Change.handleError`

@bajtos

Copy link
Copy Markdown
Member

One issue to address, the rest LGTM. No further reviews are necessary.

@bajtosbajtos assigned 0candy and unassigned bajtosMay 9, 2016
@0candy
0candyforce-pushed the remove_change_handleError branch from 31390f6 to 8ab6fccCompareMay 9, 2016 15:45
@0candy
0candy merged commit 8a6deb8 into masterMay 9, 2016
@0candy
0candy deleted the remove_change_handleError branch May 9, 2016 16:05
@0candy0candy removed the #review label May 9, 2016
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@0candy@bajtos