Skip to content

Conversation

@tastypackets
Copy link
Contributor

@tastypackets tastypackets commented Mar 1, 2024

This relates to...

Docs

Rationale

Links should work

Changes

  • Added examples to sidebar
  • Fixed broken links on home page
image

Features

N/A

Bug Fixes

N/A

Breaking Changes and Deprecations

N/A

Status

@KhafraDev KhafraDev requested a review from Ethan-Arrowood March 1, 2024 21:45
@Uzlopak Uzlopak mentioned this pull request Mar 2, 2024
7 tasks
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.81%. Comparing base (ef5fec8) to head (30a30ee).
Report is 10 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2895   +/-   ##
=======================================
  Coverage   93.81%   93.81%           
=======================================
  Files          85       85           
  Lines       23428    23428           
=======================================
  Hits        21979    21979           
  Misses       1449     1449           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Uzlopak
Uzlopak previously approved these changes Mar 3, 2024
@Uzlopak
Copy link
Contributor

Uzlopak commented Mar 5, 2024

So?

@Uzlopak
Copy link
Contributor

Uzlopak commented Mar 5, 2024

@Ethan-Arrowood Can we move the examples link to a much less dominant position?

@Ethan-Arrowood
Copy link
Collaborator

Doesn't matter to me one way or another. I'm hoping to change our doc site sooner than later anyways.

Copy link
Contributor

@Uzlopak Uzlopak left a comment

Choose a reason for hiding this comment

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

@Ethan-Arrowood
Can you approve and merge please?

Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@mcollina mcollina merged commit 03a2d43 into nodejs:main Mar 5, 2024
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.

6 participants