Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Latte 3 support #106

Merged
merged 1 commit into from
Jul 14, 2022
Merged

Latte 3 support #106

merged 1 commit into from
Jul 14, 2022

Conversation

MartinMystikJonas
Copy link
Member

@MartinMystikJonas MartinMystikJonas commented Jul 13, 2022

  • added support for Latte 3 (Latte\TranslatorExtension)
  • compatibility with Latte 2 maintained
  • with Latte 3 it is not possible to use pair {_}{/_} replaced with {translate}{/translate} (same as with Latte default translator)
  • simplified generated PHP code from tags
  • added PHPStan checks for lowest dependencies
  • latte temp variables uses prefix $ᴛ_ (Because Latte 3 forbids prefix $ʟ_)

@aleswita
Copy link
Member

aleswita commented Jul 14, 2022

@MartinMystikJonas what you think about drop support for Latte < 3?

@MartinMystikJonas
Copy link
Member Author

MartinMystikJonas commented Jul 14, 2022

@aleswita At this point it is not necessary so I would keep support for both. Latte 3 is not yet supported by all packages so many projects would keep Latte 2 for some time. We can drop support for Latte 2 later to clean up code (together with deprecated I... interfaces).

@aleswita
Copy link
Member

@MartinMystikJonas ok. Last think, can you fix last error in phpstan?

@MartinMystikJonas
Copy link
Member Author

I will look at it. For some reason prefer lowest installed different versions of libs on my machine so this error did not showed up. But it could be simply ignored because it is caused by Nette interface rename that is BC compatible but PHPStan cannot interpret it.

@MartinMystikJonas
Copy link
Member Author

@aleswita Could you approve me to run workflows? I pushed new version with PHPStan fix but it did not re-run.

@MartinMystikJonas
Copy link
Member Author

Weird now I have same versions of libs on local and in CI (it required --ignore-platform-reqs) but it still behaves differently. NO sure why.

I would need few iterations of trial-error to make it work in CI.

@MartinMystikJonas
Copy link
Member Author

Ha! Now it works! All checks passed in CI

@aleswita
Copy link
Member

perfect, thank you

@aleswita aleswita merged commit d24a581 into contributte:master Jul 14, 2022
spaze added a commit to spaze/sri-macros that referenced this pull request Oct 3, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants