Skip to content

Commit

Permalink
misc: Propagate titleFromHeading to parser settings
Browse files Browse the repository at this point in the history
The setting is not used anywhere just yet.

stack-info: PR: #368, branch: artempyanykh/stack/12
  • Loading branch information
artempyanykh committed Nov 23, 2024
1 parent 8cdff48 commit 8f3e2dd
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 6 deletions.
7 changes: 6 additions & 1 deletion Marksman/Config.fs
Original file line number Diff line number Diff line change
Expand Up @@ -334,7 +334,12 @@ let defaultMarkdownExtensions =

type ParserSettings = {
mdFileExt: string[]
titleFromHeading: bool
} with

static member OfConfig(config: Config) = { mdFileExt = config.CoreMarkdownFileExtensions() }
static member OfConfig(config: Config) = {
mdFileExt = config.CoreMarkdownFileExtensions()
titleFromHeading = config.CoreTitleFromHeading()
}

static member Default = ParserSettings.OfConfig(Config.Default)
7 changes: 2 additions & 5 deletions Marksman/Folder.fs
Original file line number Diff line number Diff line change
Expand Up @@ -501,11 +501,8 @@ module Folder =
let folderConfig = tryLoadFolderConfig folderId
let folderConfig = Config.mergeOpt folderConfig userConfig

let configuredExts =
(Option.defaultValue Config.Default folderConfig)
.CoreMarkdownFileExtensions()

let parserSettings = { mdFileExt = configuredExts }
let parserSettings =
ParserSettings.OfConfig(Option.defaultValue Config.Default folderConfig)

let documents = loadDocs parserSettings folderId

Expand Down

0 comments on commit 8f3e2dd

Please sign in to comment.