Skip to content
New issue

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

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

Already on GitHub? # to your account

Vscode debug #365

Merged
merged 5 commits into from
Apr 6, 2018
Merged

Conversation

franciscocpg
Copy link
Member

@franciscocpg franciscocpg commented Apr 6, 2018

What did you implement:

Closes #364

How did you implement it:

Adding a .vscode/launch.json to provide debug feature using vscode.

How can we verify it:

  • Checkout this PR branc
  • go to babel-webpack-4 example folder
  • Run npm install
  • Try debugging it in vscode using all the provided configurations.

Todos:

  • Write tests
  • Write documentation
  • Fix linting errors
  • Make sure code coverage hasn't dropped
  • Provide verification config / commands / resources
  • Enable "Allow edits from maintainers" for this PR
  • Update the messages below

Is this ready for review?: YES
Is it a breaking change?: NO

@@ -2,6 +2,7 @@ service: babel-webpack-4-example

# Add the serverless-webpack plugin
plugins:
- serverless-offline
Copy link
Member

@HyperBrain HyperBrain Apr 6, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this order should be reversed, shouldn't it? The sls-webpack hooks should precede the offline hooks internally.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It worked for me this way, but I realized that at README we state to use serverless-offline after serverless-webpack so I'm going to change it.

Copy link
Member

@HyperBrain HyperBrain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Thanks for the good work as always 👍

@HyperBrain HyperBrain merged commit 5cf2479 into serverless-heaven:master Apr 6, 2018
@franciscocpg franciscocpg deleted the vscode-debug branch April 6, 2018 13:57
jamesmbourne pushed a commit to jamesmbourne/serverless-webpack that referenced this pull request Oct 15, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

VSCode debugging
2 participants