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

Default Bundler Contributor Notes #9488

Merged
merged 15 commits into from
Jan 23, 2024
Merged

Default Bundler Contributor Notes #9488

merged 15 commits into from
Jan 23, 2024

Conversation

AGawrys
Copy link
Contributor

@AGawrys AGawrys commented Jan 14, 2024

↪️ Pull Request

To read the docs formatted => https://github.com/parcel-bundler/parcel/blob/bundler-contributor-notes/docs/DefaultBundler.md

This is meant to be a step-by-step explanation of the bundler, specifically createIdealGraph() within the DefaultBundler.js. It's meant to be low-level-ish and reference variables, structures, and constraints in terms of logic.

This should aid any future endeavors in creating a bundler, phasing out old structures, or supporting new features.

The PR contains visual aids for step-by-step examples as well as a document on manual bundling.

Before Merging

Once we merge in the source bundles PR, I will update all documents to reflect that change.

Copy link
Member

@devongovett devongovett left a comment

Choose a reason for hiding this comment

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

Thanks for writing these!

docs/DefaultBundler.md Outdated Show resolved Hide resolved
docs/DefaultBundler.md Outdated Show resolved Hide resolved
docs/DefaultBundler.md Outdated Show resolved Hide resolved
docs/BundlerExamples.md Outdated Show resolved Hide resolved
@AGawrys AGawrys merged commit 642d9ca into v2 Jan 23, 2024
13 of 15 checks passed
lettertwo added a commit that referenced this pull request Jan 30, 2024
* upstream/v2: (22 commits)
  Add source map support to the inline-require optimizer (#9511)
  [Web Extension] Add content script world property to manifest schema validation (#9510)
  feat: add getCurrentPackageManager (#9505)
  Default Bundler Contributor Notes (#9488)
  rename parentAsset to root for msb config and remove unstable (#9486)
  Macro errors -> v2 (#9501)
  Statically evaluate constants referenced by macros (#9487)
  Multiple css bundles in Entry bundle groups issue (#9023)
  Fix macro issues (#9485)
  Bump follow-redirects from 1.14.7 to 1.15.4 (#9475)
  Revert more CI changes to centos job (#9472)
  Use lightningcss to implement CSS packager (#8492)
  Fixup CI again (#9471)
  Clippy and use napi's Either3 (#9047)
  Upgrade to eslint 8 (#8580)
  Add support for JS macros (#9299)
  Fixup REPL CI (#9467)
  Drop per-pipeline transformation cache (#9459)
  Upgrade some CI actions (#9466)
  REPL (#9365)
  ...
@mischnic mischnic deleted the bundler-contributor-notes branch March 21, 2024 21:53
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.

2 participants