Skip to content

Add deleteLayout#281

Merged
bingenito merged 1 commit intomorganstanley:masterfrom
bingenito:ISSUE-280
Nov 14, 2019
Merged

Add deleteLayout#281
bingenito merged 1 commit intomorganstanley:masterfrom
bingenito:ISSUE-280

Conversation

@bingenito
Copy link
Member

Fixes #280

@bingenito bingenito requested a review from a team November 13, 2019 20:18
@codecov
Copy link

codecov bot commented Nov 13, 2019

Codecov Report

Merging #281 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #281      +/-   ##
==========================================
+ Coverage   93.32%   93.37%   +0.04%     
==========================================
  Files          16       16              
  Lines        1528     1539      +11     
  Branches      262      264       +2     
==========================================
+ Hits         1426     1437      +11     
  Misses        102      102
Impacted Files Coverage Δ
packages/desktopjs/src/window.ts 88.88% <ø> (ø) ⬆️
packages/desktopjs/src/container.ts 95.33% <100%> (+0.36%) ⬆️

@bingenito
Copy link
Member Author

I'm going to override the build status check to merge. It was a successful build (https://travis-ci.org/Morgan-Stanley/desktopJS/builds/611549655) and also pushed the coverage but the status isn't being updated on the PR.

@bingenito bingenito merged commit 7ff1224 into morganstanley:master Nov 14, 2019
@bingenito bingenito deleted the ISSUE-280 branch May 28, 2021 13:36
Sign up for free to 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.

Add ability to delete a layout

2 participants