Skip to content
This repository has been archived by the owner on Aug 17, 2023. It is now read-only.

Direct seeding test #226

Draft
wants to merge 18 commits into
base: main
Choose a base branch
from

Conversation

phong260702
Copy link

Pull Request Description

Address #206


Licensing Certification

FarmData2 is a Free Cultural Work and all accepted contributions are licensed as described in the LICENSE.md file. This requires that the contributor holds the rights to do so. By submitting this pull request I certify that I satisfy the terms of the Developer Certificate of Origin for its contents.

Copy link
Member

@braughtg braughtg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nicely done! I added one small suggestion regarding test robustness and readability.

Note: I did not check the tests against the issue. I'll leave that to @johnmaccormick

@nguyenbanhducA1K51 nguyenbanhducA1K51 deleted the direct_seeding_test branch April 27, 2023 19:14
@johnmaccormick
Copy link
Collaborator

@phong260702 @qnhn22 @nguyenbanhducA1K51 -- If you believe this PR is completed and ready for review, please convert it from a draft into a regular PR. Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants