Skip to content

Explain how to generate a stats.json file. - #19

Closed
karl wants to merge 1 commit into
webpack:masterfrom
karl:patch-1
Closed

Explain how to generate a stats.json file.#19
karl wants to merge 1 commit into
webpack:masterfrom
karl:patch-1

Conversation

@karl

@karl karl commented Jan 8, 2016

Copy link
Copy Markdown

No description provided.

@axe312ger

Copy link
Copy Markdown

Why a second PR for this? There is already #14

@karl

karl commented Jun 9, 2016

Copy link
Copy Markdown
Author

The PRs both do different things.

#14 adds an explanation to the docs.

This PRs adds an explanation to the UI.

@a-r-d a-r-d left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@karl Can you escape the angle bracket?

label(for="file") JSON file
input.form-control(type="file", id="file")
p You can generate a stats file using
code webpack --profile --json > stats.json

@a-r-d a-r-d Oct 26, 2016

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I believe you will want escape the > with >

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

tested this using https://naltatis.github.io/jade-syntax-docs/ and the syntax is allowed.

@evenstensberg evenstensberg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm, thanks!

@evenstensberg

Copy link
Copy Markdown
Member

@karl could you rebase?

@evenstensberg

Copy link
Copy Markdown
Member

Ups, looks like this change is already in master. thanks for the effort!

@karl
karl deleted the patch-1 branch August 27, 2026 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants