Skip to content

feat: move log parser to own package#718

Merged
lcottercertinia merged 4 commits intocertinia:mainfrom
lukecotter:feat-move-log-parser
Feb 6, 2026
Merged

feat: move log parser to own package#718
lcottercertinia merged 4 commits intocertinia:mainfrom
lukecotter:feat-move-log-parser

Conversation

@lukecotter
Copy link
Copy Markdown
Contributor

📝 PR Overview

Moves the log parser to a folder at root next to lana and log-viewer.
This is so it can be used from lana and log-viewer
Prepares it for moving into own repo .

🧩 Type of change (check all applicable)

  • 🐛 Bug fix - something not working as expected
  • ✨ New feature – adds new functionality
  • ♻️ Refactor - internal changes with no user impact
  • ⚡ Performance Improvement
  • 📝 Documentation - README or documentation site changes
  • 🔧 Chore - dev tooling, CI, config
  • 💥 Breaking change

📷 Screenshots / gifs / video [optional]

Show off your UI changes.

🔗 Related Issues

fixes #
resolves #
closes #
related #446
related #564

✅ Tests added?

  • 👍 yes
  • 🙅 no, not needed
  • 🙋 no, I need help

📚 Docs updated?

  • 🔖 README.md
  • 🔖 CHANGELOG.md
  • 📖 help site
  • 🙅 not needed

Anything else we need to know? [optional]

This is prep to move to own repo.
Will allow for reuse between log-viewer and lana vscode folder.
- Added apex-log-parser as a project in jest configuration.
- Updated log-viewer package.json to include apex-log-parser as a workspace dependency.
- Refactored imports in log-viewer components and tests to use apex-log-parser instead of local paths.
- Adjusted build and lint scripts in the root package.json to include apex-log-parser.
- Updated pnpm workspace configuration to include apex-log-parser.
@lcottercertinia lcottercertinia merged commit a0f0069 into certinia:main Feb 6, 2026
5 checks passed
@lukecotter lukecotter deleted the feat-move-log-parser branch February 8, 2026 11:45
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