Skip to content
This repository has been archived by the owner on Aug 1, 2024. It is now read-only.

docs: Add banner message about Devstack being deprecated #1289

Merged
merged 2 commits into from
Aug 1, 2024

Conversation

sarina
Copy link
Contributor

@sarina sarina commented Aug 1, 2024

@sarina
Copy link
Contributor Author

sarina commented Aug 1, 2024

@robrap - I don't think docs run on devstack builds, but does this banner wording work for you?

@robrap
Copy link
Contributor

robrap commented Aug 1, 2024

@robrap - I don't think docs run on devstack builds,

What does this mean? That you don’t think you’ll get the banner? Other?

but does this banner wording work for you?

Thank you. This should imply what people need to do, but if you are open to it, you could add a really exploit note:

For any fork of devstack, you need to look elsewhere for up-to-date docs.

I’m ok if you think this is overkill, but otherwise, it just makes the seemingly obvious very explicit.

@sarina
Copy link
Contributor Author

sarina commented Aug 1, 2024

I like the additional wording, thanks.

I was looking through the CI tests, and I don't see an automated CI test for docs. I am in a low bandwidth environment and can't do a clone of this repo, so I can't build the docs myself.

docs/conf.py Outdated Show resolved Hide resolved
Copy link
Contributor

@robrap robrap left a comment

Choose a reason for hiding this comment

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

I see. You just mean you can’t test locally? Are you willing to merge and see what happens?

@sarina
Copy link
Contributor Author

sarina commented Aug 1, 2024

I am so willing! haha

@sarina sarina merged commit 28f6d7e into master Aug 1, 2024
9 of 11 checks passed
@sarina sarina deleted the devstack-docs-banner branch August 1, 2024 12:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants