Skip to content

Conversation

@natemoo-re
Copy link
Member

Changes

  • Implements the Recursive Components RFC
  • Also implements better naming scheme for components, where the name will be generated from the file pathname if possible.

Testing

WASM test added, Astro.self added to test output.

Docs

Needs docs.

@changeset-bot
Copy link

changeset-bot bot commented Jan 27, 2022

🦋 Changeset detected

Latest commit: 36c6a4b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@astrojs/compiler Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

@jonathantneal jonathantneal left a comment

Choose a reason for hiding this comment

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

I’m always confused by opts.opts but this looks good. L32 of printer_test.go tells me what I expect. :)

@natemoo-re
Copy link
Member Author

Totally agree, @jonathantneal! Would be great to clean that up.

Copy link
Contributor

@boehs boehs left a comment

Choose a reason for hiding this comment

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

From a code quality standpoint this is spot on!

Copy link
Contributor

@matthewp matthewp left a comment

Choose a reason for hiding this comment

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

lgtm

@natemoo-re natemoo-re merged commit 02d41a8 into main Jan 27, 2022
@natemoo-re natemoo-re deleted the feat/astro-self branch January 27, 2022 23:21
@boehs boehs mentioned this pull request Jan 28, 2022
natemoo-re added a commit that referenced this pull request Jan 31, 2022
natemoo-re added a commit that referenced this pull request Jan 31, 2022
* feat: implement Astro.self (#270)

* feat: add parser

* chore: lint

* Update lib/compiler/README.md

Co-authored-by: Jonathan Neal <jonathantneal@hotmail.com>

Co-authored-by: Jonathan Neal <jonathantneal@hotmail.com>
natemoo-re added a commit that referenced this pull request Jan 31, 2022
* feat: implement Astro.self (#270)

* feat: add parser

* chore: lint

* Update lib/compiler/README.md

Co-authored-by: Jonathan Neal <jonathantneal@hotmail.com>

Co-authored-by: Jonathan Neal <jonathantneal@hotmail.com>
natemoo-re added a commit that referenced this pull request Feb 15, 2022
* feat: implement Astro.self (#270)

* feat: add parser

* chore: lint

* Update lib/compiler/README.md

Co-authored-by: Jonathan Neal <jonathantneal@hotmail.com>

Co-authored-by: Jonathan Neal <jonathantneal@hotmail.com>
natemoo-re added a commit that referenced this pull request Feb 16, 2022
* feat: implement Astro.self (#270)

* feat: add parser

* chore: lint

* Update lib/compiler/README.md

Co-authored-by: Jonathan Neal <jonathantneal@hotmail.com>

Co-authored-by: Jonathan Neal <jonathantneal@hotmail.com>
natemoo-re added a commit that referenced this pull request Feb 22, 2022
* feat: implement Astro.self (#270)

* feat: add parser

* chore: lint

* Update lib/compiler/README.md

Co-authored-by: Jonathan Neal <jonathantneal@hotmail.com>

Co-authored-by: Jonathan Neal <jonathantneal@hotmail.com>
natemoo-re added a commit that referenced this pull request Feb 22, 2022
* feat: implement Astro.self (#270)

* feat: add parser

* chore: lint

* Update lib/compiler/README.md

Co-authored-by: Jonathan Neal <jonathantneal@hotmail.com>

Co-authored-by: Jonathan Neal <jonathantneal@hotmail.com>
natemoo-re added a commit that referenced this pull request Feb 24, 2022
* feat: implement Astro.self (#270)

* feat: add parser

* chore: lint

* Update lib/compiler/README.md

Co-authored-by: Jonathan Neal <jonathantneal@hotmail.com>

Co-authored-by: Jonathan Neal <jonathantneal@hotmail.com>
natemoo-re added a commit that referenced this pull request Mar 11, 2022
* chore: enter pre

* Treat all documents as fragments which generate no implicit tags (#264)

* feat: remove "as", treat all documents as fragments with no implicit tags

* test: fix failing tests

* lint: remove unused utils

* [DO NOT MERGE] add `renderHead` util to output (#265)

* feat: add renderHead util to output

* chore: update ci to run on `next`

* test: update tests

* chore: add changeset

* fix transform output

* Add `parse` function, emit AST (#274)

* feat: implement Astro.self (#270)

* feat: add parser

* chore: lint

* Update lib/compiler/README.md

Co-authored-by: Jonathan Neal <jonathantneal@hotmail.com>

Co-authored-by: Jonathan Neal <jonathantneal@hotmail.com>

* chore: add changeset

* test: update tests for no implicit tags

* test: update tests for no implicit tags

* chore: update tests for no implicit tags

* chore: update build script

* chore: update node bindings

* chore: update wasm file

* feat: expose AST types

* chore: update tests for no implicit tags

* [ci] release (next) (#288)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* chore: update exposed types

* [ci] release (next) (#304)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* Warn when using an expression on hoist `src` attr (#303)

* Warn when using an expression on hoist `src` attr

* Adds a changeset

* Update the warning based on feedback

* [ci] release (next) (#305)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* fix: export `./types`

* [ci] release (next) (#306)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* fix: export all shared types

* fix: export named types

* [ci] release (next) (#307)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* Client only metadata (#308)

* Include metadata on client:only components

* Adds a changeset

* [ci] release (next) (#309)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* Fix next `head` behavior (#310)

* feat: update WASM tests

* fix: prefer astro.Parse to astro.ParseFragment

* fix: prefer `Parse` to `ParseFragment`

* fix: update parse test

* chore: add changeset

* test: add parse-ii test

* [ci] release (next) (#311)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* Add `fragment` to AST (#312)

* feat: add fragment node to AST output

* fix: edge case with Fragment parsing in <head>

* chore: add changeset

* chore(lint): fix lint

* chore(lint): fix lint

* [ci] release (next) (#313)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* Improve `renderHead` behavior (#315)

* feat: improve render-head behavior

* chore: add changeset

* chore: fix wasm test

* [ci] release (next) (#317)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* Refactor WASM tests (#316)

* refactor: switch tests to uvu

* chore(ci): update test script

* chore(lint): fix

* test: enable head placeholder test

* test: fix head placeholder test

* refactor: remove escapeHTML usage, fix slot in head behavior

* [ci] release (next) (#320)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* fix: failing test

* feat: expose fragment to AST (#314)

Co-authored-by: Jonathan Neal <jonathantneal@hotmail.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Matthew Phillips <matthew@skypack.dev>
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.

5 participants