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

chore(lorempixel.avatar): deprecate #1069

Merged
merged 4 commits into from
Jun 16, 2022

Conversation

import-brain
Copy link
Member

Listed as a todo in #1044

Deprecates lorempixel.avatar() in favor of internet.avatar()

@import-brain import-brain added c: chore PR that doesn't affect the runtime behavior p: 1-normal Nothing urgent labels Jun 14, 2022
@import-brain import-brain added this to the v7 - Current Major milestone Jun 14, 2022
@import-brain import-brain requested a review from a team June 14, 2022 00:18
@import-brain import-brain self-assigned this Jun 14, 2022
@import-brain import-brain requested a review from a team as a code owner June 14, 2022 00:18
@codecov
Copy link

codecov bot commented Jun 14, 2022

Codecov Report

❗ No coverage uploaded for pull request base (main@86bb94e). Click here to learn what that means.
The diff coverage is 100.00%.

❗ Current head 345028a differs from pull request most recent head 7d99ee6. Consider uploading reports for the commit 7d99ee6 to get more accurate results

@@           Coverage Diff           @@
##             main    #1069   +/-   ##
=======================================
  Coverage        ?   99.64%           
=======================================
  Files           ?     2135           
  Lines           ?   229170           
  Branches        ?      978           
=======================================
  Hits            ?   228365           
  Misses          ?      784           
  Partials        ?       21           
Impacted Files Coverage Δ
src/modules/image/providers/lorempixel.ts 91.94% <100.00%> (ø)

ST-DDT
ST-DDT previously approved these changes Jun 14, 2022
@ST-DDT ST-DDT added the s: accepted Accepted feature / Confirmed bug label Jun 14, 2022
Copy link
Member

@Shinigami92 Shinigami92 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please revert test/image.spec.ts due to we don't need direct deprecation tests anymore
@ST-DDT wrote a test that checks every @deprecated and @example JSDoc tags automatically 🚀

@import-brain import-brain dismissed stale reviews from xDivisionByZerox and ST-DDT via 0bf07dc June 14, 2022 15:42
@import-brain import-brain requested a review from Shinigami92 June 14, 2022 15:42
Shinigami92
Shinigami92 previously approved these changes Jun 14, 2022
ST-DDT
ST-DDT previously approved these changes Jun 14, 2022
@ST-DDT ST-DDT requested review from a team June 14, 2022 19:45
Shinigami92
Shinigami92 previously approved these changes Jun 15, 2022
@xDivisionByZerox
Copy link
Member

Is something blocking the merge of this?

@ST-DDT
Copy link
Member

ST-DDT commented Jun 16, 2022

Is something blocking the merge of this?

Only missing free time.^^

@Shinigami92 Shinigami92 dismissed stale reviews from xDivisionByZerox and themself via ace5b06 June 16, 2022 17:21
@Shinigami92 Shinigami92 enabled auto-merge (squash) June 16, 2022 17:40
@Shinigami92 Shinigami92 merged commit 63a37b8 into main Jun 16, 2022
@Shinigami92 Shinigami92 deleted the chore/lorempixel/deprecate-avatar branch June 16, 2022 17:44
Minozzzi pushed a commit to Minozzzi/faker that referenced this pull request Jul 19, 2022
@xDivisionByZerox xDivisionByZerox added the m: image Something is referring to the image module label Jul 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: chore PR that doesn't affect the runtime behavior m: image Something is referring to the image module p: 1-normal Nothing urgent s: accepted Accepted feature / Confirmed bug
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants