solgenomics / sgn

The code behind the Sol Genomics Network, Cassavabase and other Breedbase websites
https://solgenomics.net
MIT License
66 stars 35 forks source link

fix multi trial upload error reporting #5068

Closed Sri-2023 closed 2 months ago

Sri-2023 commented 2 months ago

Description

this will fix: #5067

Checklist

lukasmueller commented 2 months ago

Hi David, I looked at the PR with Sri and it seems that the errors are getting reported correctly.

dwaring87 commented 2 months ago

Yes, most of these checks are also done earlier in the workflow before the perl script is called. Maybe I'm overthinking that the errors found in the script should be returned by the script... I'll approve the PR