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

Merge reference analysis #367

Merged
merged 9 commits into from
May 16, 2023
Merged

Merge reference analysis #367

merged 9 commits into from
May 16, 2023

Conversation

dmail
Copy link
Member

@dmail dmail commented May 16, 2023

Analysis of inline content for js and HTML got merged into a single plugin

  • It makes more sense
  • One parsing needed instead of 2 (better performances(
  • Easier to maintain as things are regrouped together

@github-actions
Copy link

github-actions bot commented May 16, 2023


Error: Error while trying to collect info after merging merge_reference_analysis into 36.0.0.

Error: Command failed: git fetch --no-tags --prune origin merge_reference_analysis
fatal: couldn't find remote ref merge_reference_analysis

    at ChildProcess.exithandler (node:child_process:419:12)
    at ChildProcess.emit (node:events:512:28)
    at maybeClose (node:internal/child_process:1098:16)
    at Socket. (node:internal/child_process:456:11)
    at Socket.emit (node:events:512:28)
    at Pipe. (node:net:332:12)

Generated by @jsenv/file-size-impact during file size impact#4991881416 on addc886

@github-actions
Copy link

github-actions bot commented May 16, 2023


Error: Error while trying to collect info after merging merge_reference_analysis into 36.0.0.

Error: Command failed: git fetch --no-tags --prune origin merge_reference_analysis
fatal: couldn't find remote ref merge_reference_analysis

    at ChildProcess.exithandler (node:child_process:419:12)
    at ChildProcess.emit (node:events:512:28)
    at maybeClose (node:internal/child_process:1098:16)
    at Socket. (node:internal/child_process:456:11)
    at Socket.emit (node:events:512:28)
    at Pipe. (node:net:332:12)

Generated by @jsenv/performance-impact during performance_impact#4991881425 on addc886

@dmail dmail force-pushed the merge_reference_analysis branch from 029ca9f to f929ff8 Compare May 16, 2023 09:51
@dmail dmail marked this pull request as ready for review May 16, 2023 12:29
@dmail dmail merged commit bbc41cb into 36.0.0 May 16, 2023
@dmail dmail deleted the merge_reference_analysis branch May 16, 2023 12:32
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.

1 participant