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

Move to single directory and auto generate/publish docs #1246

Open
sjsrey opened this issue Jan 30, 2022 · 24 comments
Open

Move to single directory and auto generate/publish docs #1246

sjsrey opened this issue Jan 30, 2022 · 24 comments
Assignees
Labels

Comments

@sjsrey
Copy link
Member

sjsrey commented Jan 30, 2022

Currently we build the html docs out of docsrc and use a sync script to copy over to docs and publish via gh-pages:

image

It would be great to refactor this to use the gh-pages branch approach to publish and build out of a single directory.

I'm assuming this would enable the bot action to work as intended.

@sjsrey sjsrey changed the title Move to single directory and auto doc generation Move to single directory and auto generate/publish docs Jan 30, 2022
@sjsrey sjsrey added the Docs label Jan 30, 2022
@jGaboardi
Copy link
Member

I can see that this action is running, but where is it running from? The workflow is not showing up in the .github/workflows/ directory:

Screen Shot 2022-01-30 at 2 04 28 PM

@jGaboardi
Copy link
Member

Ahhh maybe because I don't have access to Settings.

@sjsrey
Copy link
Member Author

sjsrey commented Jan 30, 2022

Yup, that is the mystery.

@sjsrey
Copy link
Member Author

sjsrey commented Jan 30, 2022

I'm seeing the same thing you are:

image

@jGaboardi
Copy link
Member

In spaghetti I have access to Settings/Pages, but not for PySAL. Do you see something like this?

Screen Shot 2022-01-30 at 2 12 49 PM

@jGaboardi
Copy link
Member

Let's @knaaptime and @martinfleis in on this for potential insight.

@sjsrey
Copy link
Member Author

sjsrey commented Jan 30, 2022

yes, see the top of this issue.

@sjsrey
Copy link
Member Author

sjsrey commented Jan 30, 2022

I think the reason the docs were not getting updated (2.4 was the last version) is that I forgot to run the sync step last release. I just did a pr that has the sync #1247 which should update now.

But moving forward it would be nice to move away from this two-step docsrc -> docs dance.

@knaaptime
Copy link
Member

But moving forward it would be nice to move away from this two-step docsrc -> docs dance.

we solved this in the other packages, so this is a straightforward fix

@sjsrey
Copy link
Member Author

sjsrey commented Jan 30, 2022

The other complication I want to address is that with the move to versioneer, I'm not quite sure how to have the right tag on the docs when we build them. This is probably another good reason to automate the doc build.

@knaaptime
Copy link
Member

I'm not quite sure how to have the right tag on the docs when we build them.

martin might have some insight. Ive seen this raised over in geopandas too

@sjsrey
Copy link
Member Author

sjsrey commented Jan 30, 2022

the release is ready to go, but I wanted to see if we could dial in the doc build as part of the release. Ideally, we want to be able to update the docs if needed, but have the tag match the release tag.

@martinfleis
Copy link
Member

The other complication I want to address is that with the move to versioneer, I'm not quite sure how to have the right tag on the docs when we build them.

Tag the repository and run sphinx. That way you get a proper version on the docs. No commit in between.

@sjsrey
Copy link
Member Author

sjsrey commented Jan 30, 2022

This is the problem with the current approach:
image

@martinfleis
Copy link
Member

Yeah, because in the branch you are using, the latest tag is 2.2.0. And then there's 215 commits after that.

@sjsrey
Copy link
Member Author

sjsrey commented Jan 30, 2022

I have a draft release on gh that will create the v2.6.0 tag when published.

@sjsrey
Copy link
Member Author

sjsrey commented Jan 30, 2022

So if I publish the release, I can then update the docs and it should grab the new tag?

@martinfleis
Copy link
Member

So if I publish the release, I can then update the docs and it should grab the new tag?

Yes, that is the theory and my experience. We can always fix docs later if that won't work as expected.

@sjsrey
Copy link
Member Author

sjsrey commented Jan 30, 2022

release action appears to work for pypi

@sjsrey
Copy link
Member Author

sjsrey commented Jan 30, 2022

Will have to fix the version tag before pushing to pypi in that action:

image

Version is 2.6 but our badge in the description is 2.5.

@martinfleis
Copy link
Member

Version is 2.6 but our badge in the description is 2.5.

Try reloading, I see 2.6.0

This was referenced Jan 30, 2022
@jGaboardi
Copy link
Member

Might be another chance to look into #1216?

@sjsrey
Copy link
Member Author

sjsrey commented Jan 30, 2022

Once #1252 is merged, the remaining thing (I think) here is to add this gh-action.

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

No branches or pull requests

4 participants