You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
To support consistent development and reduce the likelihood of regressions, we propose introducing automated software tests into the project. Testing allows us to verify that new changes do not break existing functionality and provides confidence during code updates and refactoring.
We recommend integrating testing frameworks where appropriate (e.g., pytest) alongside continuous integration (CI) using GitHub Actions. This setup would automatically run the test suite on each pull request or push to key branches, ensuring that all contributions meet a baseline level of functionality before being merged. This came up as a result of discussions about #40 .
"Minimum viable testing" here I feel would include:
Ability to install the development environment locally (perhaps extending into infrastructure as code, due to the involvement of a Postgres-based implementation)
Ability to install the development environment within GitHub Actions (ideally leveraging any tooling meant for local development installation)
Existence of at least one test, processed through the development environment, that ingests a simple project and runs through MOSS to completion without exception. (For ex. ingest one repo and run a simple select a table to show that data was gathered).
An automated GitHub Actions workflow which runs tests, triggered by: 1) new or updated PR's to show whether the tests pass / fail, 2) updates to main to reaffirm the "production" branch is in good working order.
I recommend that each of these changes are applied through a single pull request to help isolate the changes and keep complexity of development + review low. Please also include documentation related to each change so that we build up our ability to understand the context together and enable onboarding for new contributors rapidly.
The text was updated successfully, but these errors were encountered:
Uh oh!
There was an error while loading. Please reload this page.
To support consistent development and reduce the likelihood of regressions, we propose introducing automated software tests into the project. Testing allows us to verify that new changes do not break existing functionality and provides confidence during code updates and refactoring.
We recommend integrating testing frameworks where appropriate (e.g.,
pytest
) alongside continuous integration (CI) using GitHub Actions. This setup would automatically run the test suite on each pull request or push to key branches, ensuring that all contributions meet a baseline level of functionality before being merged. This came up as a result of discussions about #40 ."Minimum viable testing" here I feel would include:
main
to reaffirm the "production" branch is in good working order.I recommend that each of these changes are applied through a single pull request to help isolate the changes and keep complexity of development + review low. Please also include documentation related to each change so that we build up our ability to understand the context together and enable onboarding for new contributors rapidly.
The text was updated successfully, but these errors were encountered: