twigphp / Twig

Twig, the flexible, fast, and secure template language for PHP
https://twig.symfony.com/
BSD 3-Clause "New" or "Revised" License
8.06k stars 1.24k forks source link

[RFC] deprecate tag `include` in favor of function `include` #4067

Open 94noni opened 2 months ago

94noni commented 2 months ago

hey 👋🏻

Regarding this PR and the comments/twigs/docs linked (listed down below)

I would like to know what is the best way to deprecate the tag in this root repo. I am open to work on it, just wanted to know the proper way to make review easy

thank you


friendly ping @VincentLanglet

VincentLanglet commented 2 months ago

Regarding this PR and the comments/twigs/docs linked (listed down below)

The TwigCsFixer is not directly related to the twig repository, I decide nothing about it ^^.

But some PR can be quoted as showing some direction the twig repository is taking about the include tag:

94noni commented 2 months ago

I decide nothing about it ^^

👍🏻 of course, just pointing out our discussion which leads to this issue ;)