Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Revert "Merge pull request #13504 from jasonrsadler/gen-backup-remind" #14139

Closed
wants to merge 1 commit into from
Closed

Revert "Merge pull request #13504 from jasonrsadler/gen-backup-remind" #14139

wants to merge 1 commit into from

Conversation

bsclifton
Copy link
Member

Revert #13504 in master only

Already reverted in 0.22.x-release3 and 0.23.x (see #13504 (comment))

Auditors: @jasonrsadler

@bsclifton bsclifton requested a review from jasonrsadler May 15, 2018 18:34
Copy link
Contributor

@jasonrsadler jasonrsadler left a comment

Choose a reason for hiding this comment

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

++ LGTM

@bsclifton bsclifton self-assigned this May 15, 2018
@bsclifton
Copy link
Member Author

bsclifton commented May 15, 2018

looks like there are a problem with unit tests on master (preventing this from being merged)

Reverting this worked fine (lint / unit tests passed) on 0.22.x-release3 and 0.23.x. When I run npm run unittest on master, I'm getting the same 10 failures as above. Will need to look at recent PRs to find root cause

@NejcZdovc
Copy link
Contributor

@bsclifton this PR is a problem #14073

@jasonrsadler
Copy link
Contributor

@davidtemkin and @mandar-brave are good to undo revert

@bsclifton
Copy link
Member Author

Closing as (after discussion) we'd like to keep this feature

@bsclifton bsclifton closed this May 15, 2018
@bsclifton bsclifton deleted the revert-13504 branch May 15, 2018 20:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants