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

ci: Move some code out opf the docs pass so updating to docs.tng will be easier:-) #6746

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

hunger
Copy link
Member

@hunger hunger commented Nov 8, 2024

A few CI changes to have less work updating the docs scripts :-)

@hunger hunger changed the title Tobias/push nxtokssqxstw ci: Move some code out opf the docs pass so updating to docs.tng will be easier:-) Nov 8, 2024
@ogoffart
Copy link
Member

ogoffart commented Nov 8, 2024

The reason it was in the same job is because then we don't rebuild xtask several times in different job.
But if the documentation build will no longer be in xtask, then it doesn't matter.

@hunger
Copy link
Member Author

hunger commented Nov 8, 2024

It will continue to run xtasks... But also build a lot of slint for the screenshot tool it uses :-/

@hunger
Copy link
Member Author

hunger commented Nov 8, 2024

I need to think about this... the screenshot tool takes ages to build. maybe we can build that and the xtask in a separate step and just deploy it into the build steps that need it so that it only needs to run in there.

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.

2 participants