Skip to content

Conversation

@avivkeller
Copy link
Member

Looking at some issues in nodejs/node, it seems like it may be unclear for some users to identify what a "doc" issue is versus a learn article issue.

Having a template for a learn article issue might help.

Copilot AI review requested due to automatic review settings April 24, 2025 12:04
@avivkeller avivkeller requested a review from a team as a code owner April 24, 2025 12:04
@vercel
Copy link

vercel bot commented Apr 24, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nodejs-org ✅ Ready (Inspect) Visit Preview Apr 24, 2025 8:12pm

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new issue template to differentiate learn article issues from documentation issues.

  • Introduces a dedicated learn article issue template.
  • Provides structured fields for Affected URL and Issue Description.

@codecov-commenter
Copy link

codecov-commenter commented Apr 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.81%. Comparing base (0b24039) to head (61a6085).
Report is 3 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7680      +/-   ##
==========================================
+ Coverage   74.63%   74.81%   +0.17%     
==========================================
  Files          96       98       +2     
  Lines        7689     7842     +153     
  Branches      192      197       +5     
==========================================
+ Hits         5739     5867     +128     
- Misses       1948     1973      +25     
  Partials        2        2              

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@avivkeller
Copy link
Member Author

I'm away from my machine right now, but I'll push the lint fixes by the end of the day, unless someone else beats me to it.

Signed-off-by: Aviv Keller <me@aviv.sh>
Signed-off-by: Aviv Keller <me@aviv.sh>
@github-actions
Copy link
Contributor

github-actions bot commented Apr 25, 2025

Lighthouse Results

URL Performance Accessibility Best Practices SEO Report
/en 🟢 98 🟢 100 🟢 100 🟢 91 🔗
/en/about 🟢 100 🟢 100 🟢 100 🟢 91 🔗
/en/about/previous-releases 🟢 99 🟢 100 🟢 100 🟢 92 🔗
/en/download 🟢 97 🟢 100 🟢 100 🟢 91 🔗
/en/blog 🟢 100 🟢 100 🟢 96 🟢 92 🔗

@avivkeller avivkeller added this pull request to the merge queue Apr 28, 2025
Merged via the queue into main with commit 9c2357d Apr 28, 2025
16 checks passed
@avivkeller avivkeller deleted the avivkeller-patch-1 branch April 28, 2025 16:32
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