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

Require unit tests to be written for custom scripts used by pipelines #8

Open
2 tasks done
adlersantos opened this issue Apr 27, 2021 · 0 comments
Open
2 tasks done
Assignees
Labels
feature request New feature or request revision: readme Improvements or additions to the README

Comments

@adlersantos
Copy link
Member

adlersantos commented Apr 27, 2021

Description

We expect more and more pipelines to include custom scripts (e.g. transforming CSV files, reading .shp or .nc files, scraping websites, and so on) and we need to start requiring contributors to write unit tests for them.

This will help minimize issues during internal deployment (to the Public Datasets Program) that we (Googlers) could've caught further upstream.

Some items to consider are

  • Where should contributors place the test files,
  • The unit testing conventions or guides they need to follow
  • Updating the README for unit testing requirements
  • Adding all the unit tests as part of GitHub Actions

Checklist

  • I created this issue in accordance with the Code of Conduct.
  • This issue is appropriately labeled.
@adlersantos adlersantos added revision: readme Improvements or additions to the README feature request New feature or request labels Apr 27, 2021
@tswast tswast removed their assignment Mar 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request revision: readme Improvements or additions to the README
Projects
None yet
Development

No branches or pull requests

2 participants