graphql-tools/load: use debug to log timers and errors

[ldiqual]

Description

Instead of using console.time and console.error, this PR refactors @graphql-tools/load to use debug, activated via DEBUG=graphql-tools/load.

debug is a very popular package used by libraries to log messages and allow the end-user to enable/disable logs with specific prefixes. For instance, to enable debug logs in eslint, you would run DEBUG=eslint yarn eslint. This PR applies the same concept to graphql-tools/load, where as in the old code it would print debug logs for any value of DEBUG.

Fixes #4777

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

Screenshots/Sandbox (if appropriate/relevant):

Debug logs will print like this:

How Has This Been Tested?

Ran yarn test packages/load/ and made sure the logs print properly with DEBUG=graphql-tools/load

Test Environment:

• OS: macOS 13.0
• @graphql-tools/load: 7.8.6
• NodeJS: v18.12.1

Checklist:

• I have followed the CONTRIBUTING doc and the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests and linter rules pass locally with my changes
• Any dependent changes have been merged and published in downstream modules

[changeset-bot]

🦋 Changeset detected

Latest commit: 4c94363

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages

Name	Type
@graphql-tools/load	Minor
@graphql-tools/node-require	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[ldiqual]

@ardatan Sorry to bother – just want to make sure this is on your radar. Is there anything I can do to get this merged in?

[TSMMark]

Please merge this!

The giant wall of text prints AFTER the lint results — so you can't even see your own lint results without scrolling up 20 pages of spam.

Please

[adrienharnay]

Any news on this PR?

[ardatan]

Thanks for the PR, sorry for not responding so long!
But I think we can solve this without introducing a new dependency.
I merged a different PR by keeping your credit in the CHANGELOG and commit history!
Again thanks for the contribution!