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

Warn users if they have cohortextractor actions with uncompressed csv outputs #164

Open
bloodearnest opened this issue Dec 14, 2022 · 2 comments

Comments

@bloodearnest
Copy link
Member

Probably with link to documenation.

@StevenMaude
Copy link
Contributor

This could well be a useful 🩹 to help dissuade people from using plain CSV.

I haven't run enough projects to know how visible would such a warning be, in the midst of other log output? Is it likely to be seen?

Another and/or additional possibility is having the OpenSAFELY CLI more centrally audit a project, and check for antipatterns. At the moment, it's possible to have CSV output from Data Builder, which might cause similar problems in future.

@bloodearnest
Copy link
Member Author

Good point, doing it in opensafely-cli is probably better. Moving it there.

@bloodearnest bloodearnest transferred this issue from opensafely-core/cohort-extractor Dec 14, 2022
@bloodearnest bloodearnest changed the title Warn users if they use the default of uncompressed csv outputs Warn users if they have cohortextractor actions with uncompressed csv outputs Dec 14, 2022
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

No branches or pull requests

2 participants