Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use the right link for docs comment in PR #59

Merged
merged 1 commit into from
Apr 24, 2024
Merged

Use the right link for docs comment in PR #59

merged 1 commit into from
Apr 24, 2024

Conversation

PicoCentauri
Copy link
Collaborator

Fixes #45

I think the issue was that we now have several artifacts.

I adapted the code from metatensor.

Copy link
Contributor

@Luthaf Luthaf left a comment

Choose a reason for hiding this comment

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

Looks good!

@PicoCentauri
Copy link
Collaborator Author

But shouldn't there be a comment or is this only in future PRs?

@Luthaf
Copy link
Contributor

Luthaf commented Apr 12, 2024

these workflow only run if the file is already on master, so it will be used for the next PR

@DavideTisi
Copy link
Collaborator

eh, should we merge this?

@Luthaf Luthaf merged commit 94ef0c7 into main Apr 24, 2024
10 checks passed
@Luthaf Luthaf deleted the fix-comment branch April 24, 2024 13:07
@Luthaf
Copy link
Contributor

Luthaf commented Apr 24, 2024

yes! If it is still broken, we can change the file again.

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

Successfully merging this pull request may close these issues.

Comment bot for downloading the documentation seems to be broken
3 participants