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

Remove package.json testling property #88

Merged
merged 1 commit into from
Apr 30, 2020
Merged

Remove package.json testling property #88

merged 1 commit into from
Apr 30, 2020

Conversation

ryanio
Copy link
Contributor

@ryanio ryanio commented Apr 30, 2020

This is a small cleanup PR that removes the testling property in package.json since in #87 we moved to a karma test runner for browser testing.

@ryanio ryanio changed the title Remove testling property Remove package.json testling property Apr 30, 2020
@github-actions
Copy link

Coverage Status

Coverage remained the same at 90.265% when pulling bdf0902 on cleanupTestling into 4e805bb on master.

holgerd77
holgerd77 approved these changes Apr 30, 2020
View reviewed changes
Copy link
Member

@holgerd77 holgerd77 left a comment

Choose a reason for hiding this comment

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

LGTM

@holgerd77 holgerd77 merged commit 213fbb8 into master Apr 30, 2020
@holgerd77 holgerd77 deleted the cleanupTestling branch April 30, 2020 20:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@holgerd77 holgerd77 holgerd77 approved these changes

Assignees
No one assigned
Labels
PR state: merge ready
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ryanio @holgerd77