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

Add "Designs" table to the documentation #1683

Closed
wants to merge 9 commits into from
Closed

Conversation

yarikoptic
Copy link
Member

which would provide a summary over the designs

TODOs if decide to proceed forward

  • do fill out the table with current state of things. Closes Annotate design documents with the "state" #1675 (somewhat of an alternative to have it inside the docs, could be moved there later)
  • (later or now) improve the script to just add new rows whenever new files appear

=== Do not change lines below ===
{
 "chain": [],
 "cmd": "../../scripts/produce_md_table_of_files.py -o index.md --header \"# Design documents\\n\\n\" *-*.md",
 "exit": 0,
 "extra_inputs": [],
 "inputs": [],
 "outputs": [],
 "pwd": "doc/design"
}
^^^ Do not change lines above ^^^
=== Do not change lines below ===
{
 "chain": [],
 "cmd": "tox -e format -- scripts/produce_md_table_of_files.py",
 "exit": 0,
 "extra_inputs": [],
 "inputs": [],
 "outputs": [],
 "pwd": "."
}
^^^ Do not change lines above ^^^
=== Do not change lines below ===
{
 "chain": [],
 "cmd": "sed -e 's,\",'\"'\"',g' -i scripts/produce_md_table_of_files.py",
 "exit": 0,
 "extra_inputs": [],
 "inputs": [],
 "outputs": [],
 "pwd": "."
}
^^^ Do not change lines above ^^^
=== Do not change lines below ===
{
 "chain": [
  "e3c02283b1c29a609c9397dfe00f791de73ee9d8"
 ],
 "cmd": "../../scripts/produce_md_table_of_files.py -o index.md --header \"# Design documents\\n\\n\" *-*.md",
 "exit": 0,
 "extra_inputs": [],
 "inputs": [],
 "outputs": [],
 "pwd": "doc/design"
}
^^^ Do not change lines above ^^^
@waxlamp
Copy link
Member

waxlamp commented Nov 1, 2023

Unfortunately I think this complicates our design documentation process without really adding significant value.

@waxlamp waxlamp closed this Nov 1, 2023
@yarikoptic
Copy link
Member Author

@waxlamp - did you read the #1675 ? with this PR I tried to address it - ATM we have no clue what is implemented, what replaced etc. So even if there would be some extra burden, I think it is warranted, if not significant.

@mvandenburgh mvandenburgh deleted the enh-designs-table branch June 25, 2024 15:38
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.

Annotate design documents with the "state"
2 participants