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

Updated the doc.yml #9164

Closed
wants to merge 1 commit into from
Closed

Updated the doc.yml #9164

wants to merge 1 commit into from

Conversation

Anu-Ra-g
Copy link

- pip:
# relative to this file. Needs to be editable to be accepted.
- -e ../..
- xarray
Copy link
Collaborator

Choose a reason for hiding this comment

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

this is doing something else entirely: it installs xarray from conda-forge, and thus a released version. We don't want that, because it would make previews of the current state on this repository impossible.

Copy link
Contributor

@TomNicholas TomNicholas Jun 24, 2024

Choose a reason for hiding this comment

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

If installing from conda fixes the error you see in #9141 @Anu-Ra-g (and if the docs build correctly on main) then it indicates that the reason you're seeing errors must be caused by some local changes you've made to the xarray repo. I would suggest first ensuring that you can build the documentation with no changes (i.e. on the main branch) before seeing what breaks with your changes.

Copy link
Collaborator

Choose a reason for hiding this comment

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

in this case I believe they just didn't fetch the tags, which means setuptools-scm didn't report the right version, and cfgrib complains about that.

Copy link
Author

Choose a reason for hiding this comment

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

I would suggest first ensuring that you can build the documentation with no changes (i.e. on the main branch) before seeing what breaks with your changes.

I tried building the docs from the main branch but I'm getting the same error.

@dcherian
Copy link
Contributor

Going to close since this seems to be a usage error

@dcherian dcherian closed this Jun 30, 2024
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.

Issue while building the docs.
4 participants