Skip to content

Conversation

@Shinigami92
Copy link
Member

I sorted scripts more at the top, because IMO they are frequently needed to be read by human

@Shinigami92 Shinigami92 added p: 1-normal Nothing urgent c: infra Changes to our infrastructure or project setup labels Sep 7, 2024
@Shinigami92 Shinigami92 added this to the vAnytime milestone Sep 7, 2024
@Shinigami92 Shinigami92 self-assigned this Sep 7, 2024
@Shinigami92 Shinigami92 requested a review from a team as a code owner September 7, 2024 16:33
@netlify
Copy link

netlify bot commented Sep 7, 2024

Deploy Preview for fakerjs ready!

Name Link
🔨 Latest commit ac803ad
🔍 Latest deploy log https://app.netlify.com/sites/fakerjs/deploys/66df57a2185fdc000802c716
😎 Deploy Preview https://deploy-preview-3098.fakerjs.dev
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@codecov
Copy link

codecov bot commented Sep 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.96%. Comparing base (ca9d036) to head (ac803ad).
Report is 1 commits behind head on next.

Additional details and impacted files
@@            Coverage Diff             @@
##             next    #3098      +/-   ##
==========================================
- Coverage   99.96%   99.96%   -0.01%     
==========================================
  Files        2776     2776              
  Lines      226260   226260              
  Branches      939      587     -352     
==========================================
- Hits       226180   226178       -2     
- Misses         80       82       +2     

see 2 files with indirect coverage changes

ST-DDT
ST-DDT previously approved these changes Sep 7, 2024
@ST-DDT ST-DDT requested review from a team September 7, 2024 17:10
prisis
prisis previously approved these changes Sep 7, 2024
@xDivisionByZerox xDivisionByZerox added the needs rebase There is a merge conflict label Sep 9, 2024
@Shinigami92 Shinigami92 dismissed stale reviews from xDivisionByZerox, prisis, and ST-DDT via 0b821ad September 9, 2024 20:14
@Shinigami92 Shinigami92 removed the needs rebase There is a merge conflict label Sep 9, 2024
@Shinigami92 Shinigami92 enabled auto-merge (squash) September 9, 2024 20:18
@Shinigami92 Shinigami92 merged commit 200beb9 into next Sep 9, 2024
@Shinigami92 Shinigami92 deleted the sort-package-json branch September 9, 2024 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c: infra Changes to our infrastructure or project setup p: 1-normal Nothing urgent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants