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

brainhubeu #153

Closed
wants to merge 1 commit into from
Closed

Conversation

piotr-s-brainhub
Copy link

No description provided.

@ryhinchey
Copy link
Collaborator

Other presets are for open source packages. I’m not sure it makes sense to open up presets for companies or where the line should be drawn in general for adding presets to this file.

It might make more sense to have a .envinfo file in your project that contains the configuration you want similar tools like eslint and Babel.

@piotr-s-brainhub
Copy link
Author

@ryhinchey

We have many Open Source packages https://www.npmjs.com/search?q=%40brainhubeu%2F and as we didn't get any response on this PR, we created our own package @brainhubeu/envinfo.

@ryhinchey
Copy link
Collaborator

@piotr-s-brainhub would having a file like .envinfo in each of your projects (similar to Babel/eslint) that had your config in it help you?

@piotr-s-brainhub
Copy link
Author

@ryhinchey

I haven't checked that but the @brainhubeu/envinfo package satisfies our needs so I close this PR.

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.

2 participants