Skip to content
This repository was archived by the owner on Apr 16, 2022. It is now read-only.

Support for "php-attributes" code blocks #49

Merged
merged 1 commit into from
Sep 12, 2020
Merged

Support for "php-attributes" code blocks #49

merged 1 commit into from
Sep 12, 2020

Conversation

derrabus
Copy link
Contributor

For my work on symfony/symfony-docs#14188, I need a way to express configuration samples with PHP attributes.

For instance, I'd like to add a tab "Attributes" to the code blocks on this page: https://symfony.com/doc/current/routing.html#matching-http-methods. The change proposed with this PR enables me to do that.

Copy link

@OskarStark OskarStark left a comment

Choose a reason for hiding this comment

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

Thank you

@fabpot
Copy link
Owner

fabpot commented Sep 12, 2020

Thank you @derrabus.

@fabpot fabpot merged commit 0dd00a1 into fabpot:master Sep 12, 2020
@derrabus derrabus deleted the patch-1 branch September 12, 2020 08:34
@OskarStark
Copy link

OskarStark commented Sep 12, 2020

@derrabus would you mind create a PR at https://github.com/OskarStark/doctor-rst, too?

@derrabus
Copy link
Contributor Author

@OskarStark That GitHub action of yours will probably hit me when I submit my PR, won't it? I'll try to patch it then. 😃

@derrabus
Copy link
Contributor Author

@OskarStark The action has not failed on symfony/symfony-docs#14230. Should it have?

@OskarStark
Copy link

Hmm strange, will check it on Monday. Feel free to ping me via Slack if I forget 😄

OskarStark added a commit to symfony/symfony-docs that referenced this pull request Oct 14, 2020
This PR was merged into the 5.x branch.

Discussion
----------

Use updated dependencies

Currently I fail to build the docs in 5.x. They complain on php-attributes.

This PR make sure to use @derrabus PR fabpot/sphinx-php#49 released in 2.0.2.

This change was missing from #14230

Commits
-------

3c0372a Use updated dependencies
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants