📝 Add run=False description to from_dir

[Zethson]

We could also add it to all other from_X methods?

Fixes #439

[falexwolf]

Yes.

Just: I dislike the length of the explanation given it holds true for every constructor, in particular for ln.Artifact(..., run: bool | None = False).

So, I think we should have proper documentation across the board and re-use the docstring via {} aka @doc_args; this will be messy and inconsistent otherwise.

[falexwolf]

Re the content:

Set to False to not associate the :class:~lamindb.Artifact as :attr:ln.Run.output_artifacts. This is useful if you want to register them as :attr:ln.Run.input_artifacts instead.

The thing is that this is only really relevant if ln.track() was called; otherwise run=False merely suppresses the warning about a missing ln.track() statement.

So, we should somehow add this piece of information.

After this refactor here in your nextflow PR (laminlabs/nextflow-lamin@f447359), you'd be no longer strictly required to pass run=False to avoid lineage cycles because ln.context.run is None anyway. It's still good though to have it as you'd get said warning otherwise.