Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add debugger to the host hook to simplify the inspect #5

Closed
wants to merge 1 commit into from

Conversation

@Jack-Works
Copy link

@Jack-Works Jack-Works commented Jun 7, 2020

No description provided.

@vercel
Copy link

@vercel vercel bot commented Jun 7, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click on the icon next to each commit.

@Jack-Works
Copy link
Author

@Jack-Works Jack-Works commented Oct 3, 2020

oh, it seems like I have requested this feature month ago... @devsnek hi

@@ -27,6 +27,7 @@ addEventListener('message', ({ data }) => {

const agent = new Agent({
features: [...state.get('features')],
onDebugger() { debugger }, // eslint-disable-line no-debugger

This comment has been minimized.

@devsnek

devsnek Oct 3, 2020
Member

Suggested change
onDebugger() { debugger }, // eslint-disable-line no-debugger
onDebugger() {
debugger; // eslint-disable-line no-debugger
},
@Jack-Works
Copy link
Author

@Jack-Works Jack-Works commented Oct 3, 2020

Oh I have deleted forked repo in last few months so if I have to commit the change, I need to re-fork and open a new pr. Do you accept this without review resolved? Or I need to do it later

@devsnek devsnek closed this Oct 3, 2020
@devsnek
Copy link
Member

@devsnek devsnek commented Oct 3, 2020

pushed manually.

@Jack-Works
Copy link
Author

@Jack-Works Jack-Works commented Oct 4, 2020

Cool thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.