Skip to content

Conversation

@juanri0s
Copy link
Contributor

@juanri0s juanri0s commented Dec 22, 2025

December vulns

  • node-forge
  • glob

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions:

  1. Have you added tests for the new feature
  2. Does your submission pass tests?
  3. Have you lint your code locally prior to submission?
  4. Have you updated the docs?
    • If you added new parsing or formatting options have you added them to the docs?
    • If applicable have you added an example to the parsing or formatting docs?

Changes to Core Features:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your core changes, as applicable?
  • Have you successfully ran tests with your changes locally?

@c2fo-cibot c2fo-cibot bot added the size/XXL Denotes a PR that changes 1000+ lines label Dec 22, 2025
Copy link

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 addresses security vulnerabilities by updating Docusaurus dependencies and applies consistent Prettier formatting across the codebase. The changes include updating @Docusaurus packages to v3.9.2 to resolve security issues, along with comprehensive formatting standardization using 4-space indentation for JSON, TypeScript config files, CSS, and markdown documentation.

  • Updated Docusaurus packages from v3.8.1 to v3.9.2 to address security vulnerabilities
  • Applied consistent Prettier formatting across configuration files, documentation, and source files
  • Updated version and changelog entries for the documentation package

Reviewed changes

Copilot reviewed 29 out of 36 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
documentation/package.json Updated @docusaurus/core and @docusaurus/preset-classic to v3.9.2, bumped package version to 5.0.6
CHANGELOG.md Added entry for version 5.0.6 documenting security dependency updates
documentation/CHANGELOG.md Added entry for version 5.0.6 documenting security dependency updates
tsconfig.json Applied Prettier formatting with 4-space indentation
tsconfig.build.json Applied Prettier formatting with 4-space indentation
renovate.json Applied Prettier formatting with 4-space indentation
packages/*/tsconfig.json Applied Prettier formatting with 4-space indentation across parse, format, and fast-csv packages
package.json Applied Prettier formatting with 4-space indentation
.prettierrc Applied Prettier formatting with 4-space indentation
.commitlintrc.json Applied Prettier formatting with 4-space indentation
documentation/docs/**/*.md Applied Prettier formatting including consistent list formatting, spacing, and code formatting
documentation/src/**/*.css Applied Prettier formatting with 4-space indentation for CSS properties
examples/**/*.md Applied Prettier formatting for markdown tables and content
README.md Applied Prettier formatting for list items and content structure
contributing.md Applied Prettier formatting for list items
.github/**/*.md Applied Prettier formatting for GitHub templates

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

runs-on: ubuntu-latest
strategy:
matrix:
node-version: [18.x, 20.x, 21.x]
Copy link
Contributor Author

@juanri0s juanri0s Dec 22, 2025

Choose a reason for hiding this comment

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

note we will be dropping node 18 "support". fast-csv itself uses 20.19.6.

it technically came from a docusaurus update, so we could do better about splitting there but i think we're ok with removing 18 instead

Copy link
Contributor Author

Choose a reason for hiding this comment

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

will remove required check from status once approved.

@coveralls
Copy link

coveralls commented Dec 22, 2025

Pull Request Test Coverage Report for Build 20442354111

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 96.862%

Totals Coverage Status
Change from base Build 20146907700: 0.0%
Covered Lines: 754
Relevant Lines: 772

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XXL Denotes a PR that changes 1000+ lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants