Skip to content

Add optional 3rd argument to lookup helper#542

Merged
oformaniuk merged 5 commits into
masterfrom
stef-lookup
Feb 23, 2023
Merged

Add optional 3rd argument to lookup helper#542
oformaniuk merged 5 commits into
masterfrom
stef-lookup

Conversation

@StefH

@StefH StefH commented Feb 21, 2023

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread source/Handlebars/Helpers/LookupReturnHelperDescriptor.cs Outdated
Comment thread source/Handlebars.Test/BasicIntegrationTests.cs
@sonarqubecloud

sonarqubecloud Bot commented Feb 22, 2023

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@oformaniuk oformaniuk merged commit 50614fd into master Feb 23, 2023
@oformaniuk oformaniuk deleted the stef-lookup branch February 23, 2023 20:14
@StefH

StefH commented Feb 23, 2023

Copy link
Copy Markdown
Collaborator Author

@zjklee
Thanks for merging.
(BTW I always use Squash-Commit when merging a PR to master, else you see all the individual commits which do not add that much value and the branch tree-view looks a bit cleaner)

Any idea when a new NuGet will be released?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants