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

[rc] merge from master #20

Merged
merged 9 commits into from
May 6, 2020
Merged

[rc] merge from master #20

merged 9 commits into from
May 6, 2020

Conversation

Blackbaud-AlexKingman
Copy link
Contributor

No description provided.

* removed moment

* removed moment

* unnecessary commit

* Fix unit test to actually fail

* fixed test

* destroy fixture after each

* isolated logic to make it easier to test

* removed unused imports

* removed to throw

Co-authored-by: Alex Kingman <alex.kingman@blackbaud.me>
Co-authored-by: Blackbaud-SteveBrush <steve.brush@blackbaud.com>
Co-authored-by: Alex Kingman <alex.kingman@blackbaud.me>
@Blackbaud-AlexKingman Blackbaud-AlexKingman changed the title Master merge [rc] merge from master May 6, 2020
@@ -72,6 +95,6 @@ export class SkyLinkRecordsItemDiffComponent implements OnInit {
selected: selected[f.key] || false
};
});
});
}]);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

☝️ to avoid deprecation warnings around combineLatest.

@codecov
Copy link

codecov bot commented May 6, 2020

Codecov Report

Merging #20 into rc-4.0.0 will increase coverage by 0.18%.
The diff coverage is 100.00%.

Impacted file tree graph

@@              Coverage Diff              @@
##           rc-4.0.0       #20      +/-   ##
=============================================
+ Coverage     99.81%   100.00%   +0.18%     
=============================================
  Files            36        36              
  Lines           536       534       -2     
  Branches         85        85              
=============================================
- Hits            535       534       -1     
+ Misses            1         0       -1     
Impacted Files Coverage Δ
...public/modules/link-records/link-records.module.ts 100.00% <ø> (ø)
...s/link-records/state/fields/fields.orchestrator.ts 100.00% <ø> (ø)
...nk-records/state/selected/selected.orchestrator.ts 100.00% <ø> (ø)
...pp/public/modules/link-records/link-records-api.ts 100.00% <100.00%> (ø)
...s/link-records/link-records-item-diff.component.ts 100.00% <100.00%> (ø)
...odules/link-records/link-records-item.component.ts 100.00% <100.00%> (ø)
...lic/modules/link-records/link-records.component.ts 100.00% <100.00%> (+1.03%) ⬆️
...link-records/state/matches/matches.orchestrator.ts 100.00% <100.00%> (ø)
...link-records/state/results/results.orchestrator.ts 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c447fa5...2191e32. Read the comment docs.

@Blackbaud-SteveBrush Blackbaud-SteveBrush merged commit 55f4a71 into rc-4.0.0 May 6, 2020
@Blackbaud-SteveBrush Blackbaud-SteveBrush deleted the master-merge branch May 6, 2020 20:12
Blackbaud-SteveBrush added a commit that referenced this pull request May 22, 2020
* [rc] Initial commit (#16)

* Updated changelog/package.json for 4.0.0-rc.0 release

* [rc] merge from master (#20)

* Adding license (#17)

Added license to address blackbaud/skyux2-docs#894

* removed moment (#18)

* removed moment

* removed moment

* unnecessary commit

* Fix unit test to actually fail

* fixed test

* destroy fixture after each

* isolated logic to make it easier to test

* removed unused imports

* removed to throw

Co-authored-by: Alex Kingman <alex.kingman@blackbaud.me>
Co-authored-by: Blackbaud-SteveBrush <steve.brush@blackbaud.com>

* 3.2.1 relelase (#19)

Co-authored-by: Alex Kingman <alex.kingman@blackbaud.me>

* removed merge conflicts

* removed accidental package

* upgraded deps, removed index.ts files, minor code style updates

* fixed tests

* style fix

Co-authored-by: Denise Pena <Denise.Pena@blackbaud.me>
Co-authored-by: Alex Kingman <alex.kingman@blackbaud.me>
Co-authored-by: Blackbaud-SteveBrush <steve.brush@blackbaud.com>

* Updated changelog/package.json for 4.0.0-rc.1 release

* Upgrade deps

* PR feedback

* Add Angular 9 entry under RC

Co-authored-by: Alex <alex.kingman@blackbaud.com>
Co-authored-by: Denise Pena <Denise.Pena@blackbaud.me>
Co-authored-by: Alex Kingman <alex.kingman@blackbaud.me>
Co-authored-by: John Lyons <John.Lyons@blackbaud.com>
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.

None yet

3 participants