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.

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

Remove verbose logging #34

Closed
maniax89 opened this issue Jan 29, 2019 · 5 comments · Fixed by #37
Closed

Remove verbose logging #34

maniax89 opened this issue Jan 29, 2019 · 5 comments · Fixed by #37

Comments

@maniax89
Copy link

Is there a way to make the console.log statements only print out when babel's --verbose flag is on? Right now the console is filled with these messages for every file.

@evenchange4
Copy link
Owner

evenchange4 commented Jan 29, 2019

Please try v0.3.1 with macro configuration support. Please feel free to reopen this issue if there is any problem.

// babel-plugin-macros.config.js
module.exports = {
  'react-intl': {
    verbose: false,
  },
};

@maniax89
Copy link
Author

yes this works! thank you very much

@maniax89
Copy link
Author

@evenchange4 do you know if this config option stuff works with create-react-app yet? I can open an issue in their repo but just wanted to check - I used the syntax for the package.json approach and it worked fine until I tried to use it with CRA's embedded Babel macros.

It might be that the config stuff in babel macros is very new and the package hasn't been updated yet in CRA

@maniax89
Copy link
Author

Yes it looks like at the time of this posting, CRA is still using 2.4.4 of babel-plugin-macros where the latest version 2.4.5 has a fix for the package json approach kentcdodds/babel-plugin-macros#100 so I'll just use one of the other methods for now

@maniax89
Copy link
Author

opened facebook/create-react-app#6307 to see if maybe it will get included in the next release

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 a pull request may close this issue.

2 participants