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

Support for Sphinx substitutions #1826

Merged
merged 1 commit into from
Oct 6, 2023

Conversation

rkratky
Copy link
Collaborator

@rkratky rkratky commented Oct 6, 2023

Fixes FR-5578.

  • Adds support for variable substitutions in Sphinx docs.
  • Adds an initial substitutions.txt file with a couple of definitions.
  • Modifies a sample howto to use the variables in place of version numbers and release code names.

This should be merged after #1822 as it's based off those additions (just to keep it orderly).

@rkratky rkratky force-pushed the FR-5578_version-numbers branch from 163efba to 2005c7b Compare October 6, 2023 12:51
Copy link
Collaborator

@dbungert dbungert left a comment

Choose a reason for hiding this comment

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

Should be fine but rebase please, and note that doc/custom_conf.py is currently in conflict

@rkratky rkratky force-pushed the FR-5578_version-numbers branch from abbcb42 to 96f6164 Compare October 6, 2023 13:40
@rkratky
Copy link
Collaborator Author

rkratky commented Oct 6, 2023

Rebased, conflict fixed.

@dbungert
Copy link
Collaborator

dbungert commented Oct 6, 2023

oh, did I press merge in the wrong order? one more conflict

@rkratky
Copy link
Collaborator Author

rkratky commented Oct 6, 2023

My bad. I should've been a bit more careful :) Conflict fixed, merge pls.

@dbungert dbungert force-pushed the FR-5578_version-numbers branch from cdafe2a to f7a00e4 Compare October 6, 2023 14:58
@dbungert dbungert merged commit c302876 into canonical:main Oct 6, 2023
11 checks passed
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