Skip to content

Fix/import brace-expansion errors#642

Merged
younglim merged 8 commits intomasterfrom
fix/glob-brace-expansion-error
Mar 2, 2026
Merged

Fix/import brace-expansion errors#642
younglim merged 8 commits intomasterfrom
fix/glob-brace-expansion-error

Conversation

@younglim
Copy link
Collaborator

@younglim younglim commented Mar 2, 2026

This PR adds...

  • I've kept this PR as small as possible (~500 lines) by splitting it into PRs with manageable chunks of code
  • I've requested reviews from 1 reviewer
  • I've tested existing features (website scan, sitemap, custom flow) in both node index and cli
  • I've synced this fork with GovTechSG repo
  • I've added/updated unit tests
  • I've added/updated any necessary dependencies in package[-lock].json npm audit, portable installation on GitHub Actions

@younglim younglim merged commit 02c7bd2 into master Mar 2, 2026
1 of 2 checks passed
@guardrails
Copy link

guardrails bot commented Mar 2, 2026

⚠️ We detected 1 security issue in this pull request:

Vulnerable Libraries (1)
Severity Details
High pkg:npm/jest@30.2.0 (t) upgrade to: > 30.2.0

More info on how to fix Vulnerable Libraries in JavaScript.


👉 Go to the dashboard for detailed results.

📥 Happy? Share your feedback with us.

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.

1 participant