All of frontend.util.page-property was file based except for one line.
Moved that ns to a file-based ns and moved the line out to the more
appropriate property-handler
so it can be used outside Electron e.g. when creating graphs from CLI.
Put a symlink in the old templates/config.edn location to make new
location easier to find but it's not needed for anything to work
update to match `$ bb dev:lint`
```cljs
(defn lint
"Run all lint tasks
- clj-kondo lint
- carve lint for unused vars
- lint for vars that are too large
- lint invalid translation entries"
[]
(doseq [cmd ["clojure -M:clj-kondo --parallel --lint src --cache false"
"bb lint:carve"
"bb lint:large-vars"
"bb lang:invalid-translations"
"bb lint:ns-docstrings"]]
(println cmd)
(shell cmd)))
```
not sure if we should just add `$ bb dev:lint` since it doesn't seem to be documented anywhere
this update is a pre-req to adding docs for data validation with Malli from https://github.com/logseq/logseq/blob/dev/malli-schema%2526kondo-config/
```sh
bb dev:gen-malli-kondo-config
```
Note that it will also need to be added to `build.yaml`
https://github.com/logseq/logseq/blob/dev/malli-schema%26kondo-config/.github/workflows/build.yml
CC: @RCmerci
the link https://cljdoc.org/d/cljfmt/ will always load the most up to date documentation of cljfmt.
It should be used instead of a specific version number
- Added unit test as it is used by several handlers
- Fixed test suite failing from last commit now that a test fs is used
- Fixed test stat implementation
- Extracted out Plugin schema to make usage of it less impl dependent
- Made test.helper alias consistent
- Made async testing consistent for export-test
- Add validation for plugins.edn with malli
- Add bb task using malli schema
- Add first async test that interacts with frontend.fs
- Add test helpers to make future async testing easier
- Added to CI now that it passes
- Added no-doc for docstrings that don't add any more than what's in the
ns name or for ones where I didn't know the ns that well
- rules is already used by logseq-query and will be useful for
nbb-logseq
- Updated db jobs to run in deps/db by default. Less typing but
more importantly discourages independent libraries from depending on
external scripts.
- Moved lint-rules task into db since it is db specific now
- Also remove nbb-logseq from top-level as all nbb compatibility is in
deps.
- Parser now parses all graph files like the app does, not just pages and journals.
This required extracting another fn from repo-handler
- Add and tweak CI steps that are specific to graph-parser. All
namespaces in this library are checked for nbb compatibility
- Cleaned up parser cli API so only one fn is needed for scripts
- Tests were updated to match new parsing behavior
- large_vars.clj can run with a smaller max-line-count after only refactoring two fns
- Add docs
* Validate localStorage with spec
This will help get at the source of errors like #4706 quicker
* Fix settings bugs
Close#4679 as shortcut tooltip setting is now visible.
No need to js/alert when a setting is changed. Probably leftover bugging
* Add docs and example of reusing specs in bb task
Co-authored-by: Andelf <andelf@gmail.com>
Gabriel Horner
Konstantinos Kaloutas
Junyi Du
Bad3r
Bad3r
Phoenix Eliot
Gabriel Horner