Skip to content

feat: Add environment variable to disable warnings#112

Closed
timfish wants to merge 1 commit intonodejs:mainfrom
timfish:disable-warnings
Closed

feat: Add environment variable to disable warnings#112
timfish wants to merge 1 commit intonodejs:mainfrom
timfish:disable-warnings

Conversation

@timfish
Copy link
Copy Markdown
Contributor

@timfish timfish commented Jun 19, 2024

This PR adds a DISABLE_IITM_WARNINGS environment variable that can be set to disable the logging of warnings when we fail to wrap a module.

Copy link
Copy Markdown
Contributor

@jsumners-nr jsumners-nr left a comment

Choose a reason for hiding this comment

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

I think it would be better to use Node's built-in mechanism for this. See https://github.com/fastify/process-warning for a module that makes it easy (really just port the concept into here).

@timfish
Copy link
Copy Markdown
Contributor Author

timfish commented Jun 24, 2024

I think it would be better to use Node's built-in mechanism for this

Yep, seems like a better approach!

@timfish
Copy link
Copy Markdown
Contributor Author

timfish commented Jun 25, 2024

Closing in favour of #114

@timfish timfish closed this Jun 25, 2024
@timfish timfish deleted the disable-warnings branch July 2, 2024 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants