try main if tag does not work

[lvwerra]

The previous fix #314 ignores the fact that some repos (especially community) might not have tags. We should fall back to main in that case.

[HuggingFaceDocBuilderDev]

The documentation is not available anymore as the PR was closed or merged.

[lhoestq]

good catch ^^

Though if the user explicitly passed revision I don't think we should try main

[lvwerra]

The attribute self.revision contains what the users passed and if it's None then we don't try to go for main if the initial attempt failed:

evaluate/src/evaluate/loading.py

Line 485 in 4d61255

if self.revision is None and os.getenv("HF_SCRIPTS_VERSION", SCRIPTS_VERSION) != "main":

So I think we should never overwrite whatever the user explicitly passed. Does that make sense?

[lhoestq]

Sorry I misread your code, LGTM !