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

docs: Add banner message about Devstack being deprecated#1289

Merged
sarina merged 2 commits into
masterfrom
devstack-docs-banner
Aug 1, 2024
Merged

docs: Add banner message about Devstack being deprecated#1289
sarina merged 2 commits into
masterfrom
devstack-docs-banner

Conversation

@sarina
Copy link
Copy Markdown
Contributor

@sarina sarina commented Aug 1, 2024

@sarina
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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.

Comment thread docs/conf.py Outdated
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

sarina commented Aug 1, 2024

I am so willing! haha

@sarina sarina merged commit 28f6d7e into master Aug 1, 2024
@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