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

Rename API files to use MDX extension #1013

Merged
merged 5 commits into from
Mar 13, 2024
Merged

Rename API files to use MDX extension #1013

merged 5 commits into from
Mar 13, 2024

Conversation

Eric-Arellano
Copy link
Collaborator

API docs use Markdown components, and are about to use a lot more of them with the new API components like #1008. It's confusing to pretend they're MD files.

This PR makes things consistent, which allows us to simplify some of our scripts to only deal with MDX and IPYNB extensions.

The API docs were manually changed with this Bash snippet:

for file in docs/api/**/*.md; do
  if [ -f "$file" ]; then
    mv "$file" "${file%.md}.mdx"
  fi
done

@Eric-Arellano Eric-Arellano changed the title Rename API files to use MDX extension [wip] Rename API files to use MDX extension Mar 11, 2024
Copy link
Collaborator

@arnaucasau arnaucasau left a comment

Choose a reason for hiding this comment

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

Thanks!

@Eric-Arellano Eric-Arellano changed the title [wip] Rename API files to use MDX extension Rename API files to use MDX extension Mar 12, 2024
@Eric-Arellano Eric-Arellano merged commit 0406030 into main Mar 13, 2024
3 of 5 checks passed
@Eric-Arellano Eric-Arellano deleted the EA/bye-md branch March 13, 2024 13:03
frankharkins pushed a commit to frankharkins/documentation that referenced this pull request Jul 22, 2024
API docs use Markdown components, and are about to use a lot more of
them with the new API components like
Qiskit#1008. It's confusing to
pretend they're MD files.

This PR makes things consistent, which allows us to simplify some of our
scripts to only deal with MDX and IPYNB extensions.

The API docs were manually changed with this Bash snippet:

```bash
for file in docs/api/**/*.md; do
  if [ -f "$file" ]; then
    mv "$file" "${file%.md}.mdx"
  fi
done
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants