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.

Sign up for GitHub

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

Jump to bottom

Refactor & improve serializer #383

Merged
merged 11 commits into from Feb 15, 2022
Merged

Refactor & improve serializer #383

merged 11 commits into from Feb 15, 2022

Commits on Feb 7, 2022

  1. Assemble string from serializer

    @fb55
    fb55 committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    51a7d09 View commit details
    Browse the repository at this point in the history

  2. Remove unnecessary Serializer class

    @fb55
    fb55 committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    3cccead View commit details
    Browse the repository at this point in the history

  3. Simplify serializeDocumentTypeNode

    @fb55
    fb55 committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    a00379a View commit details
    Browse the repository at this point in the history

  4. Delay ns lookup

    @fb55
    fb55 committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    cf0b731 View commit details
    Browse the repository at this point in the history

  5. Move serialize fn to serializer file 

    and improve the options type
    @fb55
    fb55 committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    677d83a View commit details
    Browse the repository at this point in the history

  6. Add scriptingEnabled flag to serializer 

    Fixes inikulin#332
    @fb55
    fb55 committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    18f170c View commit details
    Browse the repository at this point in the history

  7. Fix serializing mixed content 

    Fixes inikulin#333
    @fb55
    fb55 committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    67dbf72 View commit details
    Browse the repository at this point in the history

  8. feat: Add serializeOuter method 

    Fixes inikulin#230, inikulin#378

Uses the test case from inikulin#378, by @joeldenning

Co-Authored-By: Joel Denning <5524384+joeldenning@users.noreply.github.com>
    @fb55 @joeldenning
    fb55 and joeldenning committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    844f072 View commit details
    Browse the repository at this point in the history

  9. Move serializer tests to data file

    @fb55
    fb55 committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    9424821 View commit details
    Browse the repository at this point in the history

  10. Add missing issue reference

    @fb55
    fb55 committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    feaf509 View commit details
    Browse the repository at this point in the history

  11. fix: serialize top-level template contents 

    Before this, passing a `template` tag to `serialize` would result in an empty string.
    @fb55
    fb55 committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    a6f4b62 View commit details
    Browse the repository at this point in the history