Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Use Doctor RST 1.41.0 and new rule TitleUnderlineLengthMustMatchTitleLength #18026

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 8, 2023
Merged

Use Doctor RST 1.41.0 and new rule TitleUnderlineLengthMustMatchTitleLength #18026

merged 1 commit into from
Mar 8, 2023

Conversation

MrYamous
Copy link
Contributor

@MrYamous MrYamous commented Mar 8, 2023

@MrYamous MrYamous requested a review from OskarStark as a code owner March 8, 2023 12:04
@carsonbot carsonbot added this to the 5.4 milestone Mar 8, 2023
@MrYamous MrYamous changed the title Use Doctor RST 1.41.0 and new rule Use Doctor RST 1.41.0 and new rule TitleUnderlineLengthMustMatchTitleLength Mar 8, 2023
@OskarStark
Copy link
Contributor

Can you please check the CI errors? Thanks

@MrYamous MrYamous requested a review from xabbuh as a code owner March 8, 2023 12:15
@MrYamous
Copy link
Contributor Author

MrYamous commented Mar 8, 2023

I guess CI for 6.2 and 6.3 will be broken too after merge

@OskarStark
Copy link
Contributor

Thank you @MrYamous.

@OskarStark OskarStark force-pushed the ci/update-doctor-rst-141 branch from e19d25f to 66e0401 Compare March 8, 2023 14:19
@OskarStark OskarStark merged commit 08e6ad8 into symfony:5.4 Mar 8, 2023
@OskarStark
Copy link
Contributor

I unmerged the PR, can you please check 6.2? Thanks

@MrYamous MrYamous mentioned this pull request Mar 8, 2023
Merged
OskarStark added a commit that referenced this pull request Mar 9, 2023
@OskarStark
minor #18027 Fix rst syntax for ci with Doctor RST 141 (MrYamous)
e4b2ae6 
This PR was merged into the 6.2 branch.

Discussion
----------

Fix rst syntax for ci with Doctor RST 141

Followup #18026

Commits
-------

0013f27 fix rst syntax for ci
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@OskarStark OskarStark OskarStark approved these changes

@xabbuh xabbuh Awaiting requested review from xabbuh xabbuh is a code owner

Assignees
No one assigned
Labels
Status: Reviewed
Projects
None yet
Milestone
5.4
Development

Successfully merging this pull request may close these issues.
3 participants
@MrYamous @OskarStark @carsonbot