Skip to content

chore(build): update Twing dependency #1206

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

Merged
merged 3 commits into from
May 24, 2020
Merged

chore(build): update Twing dependency #1206

merged 3 commits into from
May 24, 2020

Conversation

ringods
Copy link
Contributor

@ringods ringods commented May 12, 2020

Summary of changes:

  • Update Twing engine to v5.0.1

@ringods ringods requested review from bmuenzenmeyer and sghoweri May 12, 2020 18:09
@ringods ringods self-assigned this May 12, 2020
@ringods ringods added the dependencies Pull requests that update a dependency file label May 12, 2020
@bmuenzenmeyer
Copy link
Member

Does the 4 to 5 upgrade impact end users?

@ringods
Copy link
Contributor Author

ringods commented May 14, 2020

@bmuenzenmeyer I tested it locally and there were no regressions AFAICS. But one of the fixed issues regarding relative imports triggered me to test it regarding the issue I asked your help for.

@ringods
Copy link
Contributor Author

ringods commented May 24, 2020

@bmuenzenmeyer could this get merged?

I am hoping to test engine-twig with a newer twing library aiming to find the cause for #1201

@coveralls
Copy link

coveralls commented May 24, 2020

Coverage Status

Coverage remained the same at 73.411% when pulling e3d5b30 on feature/twing into 44cb7ed on dev.

@bmuenzenmeyer
Copy link
Member

bmuenzenmeyer commented May 24, 2020

I saw the other message about uneven head and foot calls. Does this engine or the twig library render as promises?

(Thanks for patience recently, I'm swamped at work)

@ringods
Copy link
Contributor Author

ringods commented May 24, 2020

Yes it does. See the section Rendering Templates from the docs: https://nightlycommit.github.io/twing/api.html

@bmuenzenmeyer bmuenzenmeyer merged commit 0711850 into dev May 24, 2020
@bmuenzenmeyer bmuenzenmeyer deleted the feature/twing branch May 24, 2020 15:13
@sghoweri
Copy link
Contributor

PR was released with v5.11.1

antonia-rose pushed a commit to quelltexterin/nemo-uikit-workshop that referenced this pull request Apr 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file released 🚀
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants