Skip to content

Conversation

@Veebaa
Copy link
Contributor

@Veebaa Veebaa commented Sep 15, 2025

Converted advanced.rst to advanced.md.

Part of issue #2886

Copy link
Contributor

@denuoweb denuoweb left a comment

Choose a reason for hiding this comment

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

switch eval-rst to native myst in two spots

docs/advanced.md Outdated

## Callbacks for Validation

```{eval-rst}
Copy link
Contributor

Choose a reason for hiding this comment

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

You can remove the eval-rst for the versionchanged:: 2.0, just like on line 415 ```{versionchanged} 8.2

docs/advanced.md Outdated

## Token Normalization

```{eval-rst}
Copy link
Contributor

Choose a reason for hiding this comment

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

You can remove the eval-rst for versionadded:: 2.0 by using ```{versionadded} 2.0


```{eval-rst}
.. currentmodule:: click
```
Copy link
Collaborator

Choose a reason for hiding this comment

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

Change to use native myst syntax: ```{currentmodule} click

@Rowlando13
Copy link
Collaborator

@denuoweb Thanks for the review. @Veebaa Just a few things to fix.

@Veebaa
Copy link
Contributor Author

Veebaa commented Sep 16, 2025

All fixed 😊

@Rowlando13 Rowlando13 merged commit f6a87df into pallets:stable Sep 22, 2025
2 checks passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants