Skip to content

[Sphinx] [Automation] Make sure Issue -> PR conversion of tables is correct #210

@felix91gr

Description

@felix91gr

In the two CERT-C ports of guidelines relating to Shifts over integer types, the table conversion from the original Markdown to Sphinx / RST didn't work well.

The PRs have since been fixed by hand. Regardless, here are the two pairs of issues and PRs that I'm talking about:

And here's an illustration of how the table looked in the original text:

  | **Compilation Mode** | **`0 <= M < N`** | **`M < 0`**         |     **`N <= M`**    |
  |:--------------------:|:----------------:|:---------------------:|:-------------------:|
  | Debug                | Shifts normally  | Panics              | Panics              |
  | Release              | Shifts normally  | Shifts by `M mod N` | Shifts by `M mod N` |

CC @x0rw lemme know if there's something I can do to start improving the table conversion (maybe we can use another conversion library?)

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingcontributor experienceRelated to making contributions easierhelp wantedExtra attention is neededquestionFurther information is requested

    Type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions