Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Watcher: cleanup ensureWatchExists use #31926

Merged
merged 5 commits into from
Jul 13, 2018
Merged

Watcher: cleanup ensureWatchExists use #31926

merged 5 commits into from
Jul 13, 2018

Commits on Jul 10, 2018

  1. Watcher: cleanup ensureWatchExists use 

    Previously, the ensureWatchExists was overridable. This commit makes
it final so that it cannot be overridden, and cleans up some redundant
code in the process.
    @hub-cap
    hub-cap committed Jul 10, 2018
    Configuration menu
    Copy the full SHA
    55c783b View commit details
    Browse the repository at this point in the history

  2. Move the call to ensure watch to the begin of the ctor

    @hub-cap
    hub-cap committed Jul 10, 2018
    Configuration menu
    Copy the full SHA
    f6e041a View commit details
    Browse the repository at this point in the history

  3. Merge remote-tracking branch 'upstream/master' into final_ensure_watch

    @hub-cap
    hub-cap committed Jul 10, 2018
    Configuration menu
    Copy the full SHA
    0c4b6f0 View commit details
    Browse the repository at this point in the history

  4. Merge remote-tracking branch 'upstream/master' into final_ensure_watch

    @hub-cap
    hub-cap committed Jul 10, 2018
    Configuration menu
    Copy the full SHA
    5f1ce34 View commit details
    Browse the repository at this point in the history

Commits on Jul 13, 2018

  1. Merge remote-tracking branch 'upstream/master' into final_ensure_watch

    @hub-cap
    hub-cap committed Jul 13, 2018
    Configuration menu
    Copy the full SHA
    645fce2 View commit details
    Browse the repository at this point in the history