Skip to content
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.

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

nette 3.1 compatibility #183

Merged
merged 18 commits into from
Feb 19, 2021
Merged

nette 3.1 compatibility #183

merged 18 commits into from
Feb 19, 2021

Conversation

vrestihnat
Copy link
Contributor

@vrestihnat vrestihnat commented Jan 29, 2021

Simple adjustments for nette 3.1 and latte 2.10.1 compatibility. Please publish changes.

@lulco
Copy link
Contributor

lulco commented Feb 4, 2021

@enumag please can you take a look to this? It fixes master and other PR will have green build too. Thanks!

composer.json Outdated Show resolved Hide resolved
.travis.yml Outdated Show resolved Hide resolved
@lulco
Copy link
Contributor

lulco commented Feb 9, 2021

any news here?

Co-authored-by: Jáchym Toušek <enumag@gmail.com>
@lulco
Copy link
Contributor

lulco commented Feb 18, 2021

@vrestihnat you are very close :) please don't give up

@vrestihnat
Copy link
Contributor Author

I don't give up. I just don't see any more comments so we can close it.

@enumag enumag merged commit 3adb0d8 into Kdyby:master Feb 19, 2021
@enumag
Copy link
Member

enumag commented Feb 19, 2021

Please check if we also need #182 and/or #179. Don't wanna make a half-baked release.

@vrestihnat
Copy link
Contributor Author

#182 is useful, it solves compatibility with pair macros. #179 is obsolete, #180 is resolved. I will prepare a pull request that contains #182 + test

@vrestihnat
Copy link
Contributor Author

#177 is partially solved and the rest is #182

nufue pushed a commit to inspire-develop/Translation that referenced this pull request Apr 27, 2022
* nette 3.1 compatibility

* latte fix max compatibility version

* fix cs

* Update .travis.yml

* latte/latte 2.10.1 compatibility

* fix if not exists latte engine constant

* clearing .travis.yml

* clearing .gitignore

* composer.json - original dependence latte/latte

* Update tests/KdybyTests/Translation/LatteExtractorTest.phpt

* useless variables in LatteExtractorTest.phpt

* CS Squiz.Strings.DoubleQuoteUsage.NotRequired was excluded from Extractor test

* Update LatteExtractorTest.phpt

* Update ruleset.xml

* Update src/DI/TranslationExtension.php

Co-authored-by: Jáchym Toušek <enumag@gmail.com>

Co-authored-by: Jáchym Toušek <enumag@gmail.com>
nufue pushed a commit to inspire-develop/Translation that referenced this pull request Nov 22, 2022
* nette 3.1 compatibility

* latte fix max compatibility version

* fix cs

* Update .travis.yml

* latte/latte 2.10.1 compatibility

* fix if not exists latte engine constant

* clearing .travis.yml

* clearing .gitignore

* composer.json - original dependence latte/latte

* Update tests/KdybyTests/Translation/LatteExtractorTest.phpt

* useless variables in LatteExtractorTest.phpt

* CS Squiz.Strings.DoubleQuoteUsage.NotRequired was excluded from Extractor test

* Update LatteExtractorTest.phpt

* Update ruleset.xml

* Update src/DI/TranslationExtension.php

Co-authored-by: Jáchym Toušek <enumag@gmail.com>

Co-authored-by: Jáchym Toušek <enumag@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants