Skip to content

Another attempt at supporting nested sections #2215

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

Draft
ArnaudLigny wants to merge 11 commits into
base: master
Choose a base branch
from

Update Section.php

9c38d8e
Select commit
Loading
Failed to load commit list.
Draft

Another attempt at supporting nested sections #2215

Update Section.php
9c38d8e
Select commit
Loading
Failed to load commit list.
Scrutinizer / Inspection succeeded Sep 27, 2025 in 5m 11s

Summary

1 new issues, 1 updated code elements

Annotations

Check notice on line 696 in src/Collection/Page/Page.php

See this annotation in the file changed.

@scrutinizer scrutinizer / Inspection

src/Collection/Page/Page.php#L696 

It seems like ``Cecil\Collection\Page\PrefixSuffix::getSuffix($filename)`` can also be of type ``null``; however, parameter ``$path`` of ``Cecil\Util::joinPath()`` does only seem to accept ``string``, maybe add an additional type check?
View more details on Scrutinizer