Skip to content

fix: t macro as function typed#821

Merged
tricoder42 merged 1 commit intolingui:mainfrom
semoal:fix--t-as-function-type
Nov 5, 2020
Merged

fix: t macro as function typed#821
tricoder42 merged 1 commit intolingui:mainfrom
semoal:fix--t-as-function-type

Conversation

@semoal
Copy link
Copy Markdown
Contributor

@semoal semoal commented Nov 5, 2020

Totally forgot the type 🤕
Will fix @Bertg comment of #807
Screenshot 2020-11-05 at 12 26 06

@semoal semoal requested a review from tricoder42 November 5, 2020 11:26
@vercel
Copy link
Copy Markdown

vercel Bot commented Nov 5, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/lingui-js/js-lingui/g0xpfvat6
✅ Preview: https://js-lingui-git-fix-t-as-function-type.lingui-js.vercel.app

@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 5, 2020

Codecov Report

Merging #821 into main will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #821   +/-   ##
=======================================
  Coverage   83.87%   83.87%           
=======================================
  Files          39       39           
  Lines        1333     1333           
  Branches      356      356           
=======================================
  Hits         1118     1118           
  Misses        129      129           
  Partials       86       86           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2d9e124...64a2f82. Read the comment docs.

@tricoder42 tricoder42 merged commit 7f09c2d into lingui:main Nov 5, 2020
@tricoder42
Copy link
Copy Markdown
Contributor

Ah, cool! Good catch 👍

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.

2 participants