Skip to content
This repository was archived by the owner on Apr 3, 2024. It is now read-only.

Start using async/await to flatten the pyramid in Debuglet.start#307

Merged
ofrobots merged 2 commits intogoogleapis:masterfrom
ofrobots:asyncify
Jul 11, 2017
Merged

Start using async/await to flatten the pyramid in Debuglet.start#307
ofrobots merged 2 commits intogoogleapis:masterfrom
ofrobots:asyncify

Conversation

@ofrobots
Copy link
Copy Markdown
Contributor

Start using async/await to simplify Debuglet.start. This exercise can be continued for the remainder of the nested callbacks in follow-on PRs. This PR peels the first two layers only.

For an easier time reviewing, review each commit individually.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 10, 2017
@ofrobots ofrobots requested a review from DominicKramer July 10, 2017 15:33
@ofrobots
Copy link
Copy Markdown
Contributor Author

Also.. adding ?w=1 to the URL makes the code a lot easier to review. Why isn't this the default on github?

Copy link
Copy Markdown
Contributor

@DominicKramer DominicKramer left a comment

Choose a reason for hiding this comment

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

LGTM w/nit

const mapFiles = fileStats.selectFiles(/.map$/, process.cwd());
SourceMapper.create(mapFiles, function(err3, mapper) {
if (err3) {
that.logger_.error('Error processing the sourcemaps.', err3);

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@ofrobots ofrobots merged commit 800ed08 into googleapis:master Jul 11, 2017
@ofrobots ofrobots deleted the asyncify branch July 11, 2017 00:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants