Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Removing debug #144

Merged
merged 5 commits into from
May 18, 2023
Merged

Removing debug #144

merged 5 commits into from
May 18, 2023

Conversation

tackaberry
Copy link

Resolves #122.

Removes debug module completely.

Reduces build size from:

166K    ./dist/cjs
41K     ./dist/esm
108K    ./dist/iife
315K    ./dist/

To:

165K    ./cjs
41K     ./esm
76K     ./iife
282K    .

@fershad
Copy link
Contributor

fershad commented May 10, 2023

@tackaberry looks good. I'll give it a proper once over early next week and look to include this in the next patch release.

@fershad
Copy link
Contributor

fershad commented May 18, 2023

Took the chance to add some JSDoc comments to the hosting files which lacked them.

@fershad fershad merged commit 1973dd8 into thegreenwebfoundation:main May 18, 2023
@fershad fershad mentioned this pull request May 18, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Move debug to devDependencies
2 participants
@tackaberry @fershad