Skip to content

Code refactoring improvements for Oobee#680

Open
younglim wants to merge 16 commits intomasterfrom
refactor-utils-extract-lifecycle-and-remove-legacy-utils
Open

Code refactoring improvements for Oobee#680
younglim wants to merge 16 commits intomasterfrom
refactor-utils-extract-lifecycle-and-remove-legacy-utils

Conversation

@younglim
Copy link
Copy Markdown
Collaborator

@younglim younglim commented Mar 12, 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 requested a review from WilsonkwSheng March 12, 2026 09:22
@younglim
Copy link
Copy Markdown
Collaborator Author

Tested E2E with Cypress and Playwright integration of Oobee

@younglim younglim closed this Mar 12, 2026
@younglim younglim reopened this Mar 17, 2026
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.

2 participants