ExtensionEngine / tailor

Content authoring platform
MIT License
31 stars 10 forks source link

🟢 Fixing eslint issues that cause the Circle-CI build failure(s) #992

Closed marjev closed 1 year ago

marjev commented 2 years ago

This PR is two-fold:

As a reminder, once the PR has been approved and merged, we should publish the new version of the core-components to npm.

@underscope please review, when you get a chance, to ensure that eslint issues have been addressed in a satisfactory manner. Some of the resolutions are a product of a gut feeling and might not be aligned with our company-wide or repo-wide coding standards/guidelines.

cc @MiroDojkic - since some of the lines I've touched have been written by you, you might want to 👀 . Don't feel obligated.

To do:

Sidenotes:

marjev commented 1 year ago

Comments addressed in b538d6b5bb0c294856958c612be13db5928da834.