Skip to content

Make it an error to unregister a locator strategy that hasn't been registered #961

Closed

Description

@pekkaklarck

Currently if you try to unregister a locator strategy that doesn't exist, there's only a notification in the log file that no such strategy exists and nothing is done. This can lead to problems if, for example, there's a typo in the strategy name and the actual strategy is accidentally not unregistered. Better make this situation an error so that users see if there has been a problem.

Activity

  1. added this to the v3.0.0 milestone on Oct 19, 2017
  2. pekkaklarck commented on Oct 19, 2017

    @pekkaklarck
    MemberAuthor

    This is implemented by f383470 and will be fixed if/when PR #956 is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions