Skip to content

Fix broken build on development#7165

Merged
iAmWillShepherd merged 2 commits intodevelopmentfrom
fix-accidental-removal-of-dispatcher-method
Mar 26, 2019
Merged

Fix broken build on development#7165
iAmWillShepherd merged 2 commits intodevelopmentfrom
fix-accidental-removal-of-dispatcher-method

Conversation

@niik
Copy link
Member

@niik niik commented Mar 26, 2019

This reverts commit c2e7276.

Overview

#7018 was an old PR that recently got merged; in it there was a commit that removed a method that was unused at the time but that was in use on development.

We should discuss turning on branch protection to require being up to date with development before merging to avoid these kinds of issues.

Release notes

Notes: no-notes

This reverts commit c2e7276.
@niik niik added the ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Mar 26, 2019
@iAmWillShepherd iAmWillShepherd merged commit aaaf77d into development Mar 26, 2019
@iAmWillShepherd iAmWillShepherd deleted the fix-accidental-removal-of-dispatcher-method branch March 26, 2019 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-review Pull Requests that are ready to be reviewed by the maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants