search

metomi / fab

Flexible build system for scientific software
https://metomi.github.io/fab/
Other
19 stars 11 forks source link

Create artefact store object #301

Closed hiker closed 2 months ago

hiker commented 2 months ago

This fixes a #TODO, and prepares for #300.

hiker commented 2 months ago

OK, I don't know what to do about the type information, please advise ;)

hiker commented 2 months ago

I imagine our future contains a long and involved discussion about redesigning and refactoring the way artefacts are stored and managed but in the mean time this tidies up the current implementation a little.

Hmm - not really, it seems to work :) The only question I have is in #300: for the follow-up PR, do you prefer to have methods for certain standard artefacts (e.g. all f90 files, i.e. all files to be compiled by Fortran, ...), or do you prefer to have some constants exposed and use them in the standard way?

Otherwise ready for next review.

hiker commented 2 months ago

Gee, python-specific error messages - annoying.