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 front matter handling and extract behavior into loaders #778

Merged
merged 4 commits into from
Feb 17, 2024
Merged

Refactor front matter handling and extract behavior into loaders #778

merged 4 commits into from
Feb 17, 2024

Commits on Aug 22, 2023

  1. Extract front matter loading into defined classes 

    This change refactors the loading of front matter into a series of
classes. This is a first step toward making front matter loaders
pluggable to allow, for example, [TOML][1]- or [KDL][2]-based front
matter.

[1]: https://toml.io
[2]: https://kdl.dev/
    @michaelherold
    michaelherold committed Aug 22, 2023
    Configuration menu
    Copy the full SHA
    e2ccac1 View commit details
    Browse the repository at this point in the history

  2. Remove unused constants 

    These were developed prior to FrontMatterImporter and are no longer used.
    @michaelherold
    michaelherold committed Aug 22, 2023
    Configuration menu
    Copy the full SHA
    e1baa67 View commit details
    Browse the repository at this point in the history

  3. Consolidate front matter code in a module 

    This change reorganizes the front matter-related code so it is fully
contained within the Bridgetown::FrontMatter module. That makes it
easier to find the functionality and standardizes the way that
Bridgetown refers to front matter.

It also adds deprecated constant proxies for backwards-compatibility.
It's unclear whether the moved modules were public API or not so the
proxies make it so updating will not break anyone's site without
warning.
    @michaelherold
    michaelherold committed Aug 22, 2023
    Configuration menu
    Copy the full SHA
    20dceb1 View commit details
    Browse the repository at this point in the history

  4. Remove unnecessary include 

    This is already done by the include of the importer so isn't necessary
at this point.
    @michaelherold
    michaelherold committed Aug 22, 2023
    Configuration menu
    Copy the full SHA
    11d328d View commit details
    Browse the repository at this point in the history