Skip to content

chore(7506): add debug setup for local om-kursen-ladok-client#344

Open
belanglos wants to merge 1 commit intomasterfrom
issues/7506-enable-debugging-of-local-oklc
Open

chore(7506): add debug setup for local om-kursen-ladok-client#344
belanglos wants to merge 1 commit intomasterfrom
issues/7506-enable-debugging-of-local-oklc

Conversation

@belanglos
Copy link
Contributor

Summary

  • add launch config
  • add start-script for debugging
  • add scripts to switch

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

Adds local debugging support for working with a local checkout of @kth/om-kursen-ladok-client, including VS Code attach configuration and helper npm scripts for starting the server in inspect mode and swapping dependency sources.

Changes:

  • Add start-dev-debug npm script to run nodemon with Node inspector enabled.
  • Add npm scripts to swap @kth/om-kursen-ladok-client between a local file: install and the published package.
  • Commit a VS Code .vscode/launch.json and adjust .gitignore to track it.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 3 comments.

File Description
package.json Adds debug/start and dependency-switch helper scripts for local development.
.vscode/launch.json Adds a VS Code “attach” debug configuration intended to work with --inspect.
.gitignore Ignores all .vscode files except launch.json so the debug config is committed.

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

You can also share your feedback on Copilot code review. Take the survey.

@belanglos belanglos force-pushed the issues/7506-enable-debugging-of-local-oklc branch from b868f58 to cb101a3 Compare March 13, 2026 13:46
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