Skip to content

Commit e1008d6

Browse files
apply code review recommendations
1 parent b3d4968 commit e1008d6

File tree

2 files changed

+23
-25
lines changed

2 files changed

+23
-25
lines changed

README.md

Lines changed: 21 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -49,11 +49,11 @@ Add metadata on your namespace to set the title, author, and tags.
4949
Configure your author profile in [clay.edn](clay.edn).
5050

5151
Images can be added to the same folder as the namespace,
52-
and displayed in the notebook with markdown like `![caption](my-image.jpg)`.
52+
and displayed with markdown like `;; ![caption](my-image.jpg)`.
5353
The first image on the page is used as a preview in the blog listing,
5454
unless a different image is listed in the metadata.
5555

56-
[Kindly](https://scicloj.github.io/kindly-noted/kindly) annotations in your notebook are rendered as visualizations.
56+
[Kindly](https://scicloj.github.io/kindly-noted/kindly) annotations are rendered as visualizations.
5757

5858
```clojure
5959
^kind/table
@@ -65,11 +65,11 @@ unless a different image is listed in the metadata.
6565

6666
## Preview the Website **(Optional)**
6767

68-
[Set up your editor with Clay REPL commands](https://scicloj.github.io/clay/#setup) to visualize the notebook as you write.
68+
[Use Clay REPL commands](https://scicloj.github.io/clay/#setup) to visualize the namespace as you write.
6969

7070
Building is delegated to Clay and Quarto.
7171

72-
When using Clay interactively it render directly to HTML for speed.
72+
When using Clay interactively it renders directly to HTML for speed.
7373
The published site goes through a longer process of producing markdown then HTML.
7474

7575
```sh
@@ -94,11 +94,11 @@ Create a pull request:
9494
Your pull request will be reviewed to prevent abuse and then merged.
9595
Once merged, namespaces are automatically published to the website.
9696

97-
Please contact [@timothypratley](https://github.com/timothypratley) if you are having any difficulty submitting a notebook.
97+
Please contact [@timothypratley](https://github.com/timothypratley) if you are having any difficulty submitting a PR.
9898

9999
### Check Your Page Views
100100

101-
Publicly available [page view analytics](https://clojurecivitas.goatcounter.com/) indicate how widely your notebook is being shared.
101+
Publicly available [page view analytics](https://clojurecivitas.goatcounter.com/) indicate how widely your idea is being shared.
102102

103103
### Editing the Civitas Explorer Database
104104

@@ -111,32 +111,29 @@ See the [explorer](https://clojurecivitas.github.io/civitas/explorer.html).
111111

112112
Align with Clojure's values: simplicity, community, and tooling that helps you think.
113113

114-
### Notebook Namespace Selection
114+
### Namespace Selection
115115

116-
A notebook’s namespace serves as a clear, unique path to its content and follows **Clojure’s naming conventions**.
116+
A namespace serves as a clear, unique path to its content and follows **Clojure’s naming conventions**.
117117

118-
The namespace should emphasize **what the notebook is about**, not how it is categorized.
118+
The namespace should emphasize **what the narrative is about**, not how it is categorized.
119119
Think of it as a logical path that leads to a specific artifact or topic.
120120
Classification elements such as tags, author, document type, level, or publication date belong in **metadata**, not the
121121
namespace.
122122

123-
- **Start with an organization** if the notebook is about a library or tool maintained by one.
123+
- **Start with an organization** if the narrative is about a library or tool maintained by one.
124124
Examples: `scicloj`, `lambdaisland`.
125125
- **Follow with the specific library or concept.**
126-
Examples: `scicloj.clay`, `lambdaisland.kaocha`.
126+
Examples: `scicloj.clay`, `lambdaisland.kaocha`, `lambdaisland.hiccup`.
127127
- If there is **no organization**, start directly with the library or tool name.
128128
Examples: `hiccup`, `reagent`.
129-
If the notebook refers to an alternative implementation, **include the organization** to disambiguate.
130-
Example: `lambdaisland.hiccup`.
131129
- For **core Clojure topics**, use `clojure` as the root.
132130
Examples: `clojure.lazy-sequences`, `clojure.transducers`.
133-
- Add **segments** to describe the notebook’s content. These segments should:
131+
- Add **segments** to further qualify the namespace. These segments should:
134132
- Avoid name collisions.
135133
- Not duplicate metadata.
136-
- The last segment should be specific and descriptive. Prefer: `z-combinator-gambit`, avoid: `z-combinator`.
134+
- The last segment should be extra specific and descriptive. Prefer: `z-combinator-gambit`, avoid: `z-combinator`.
137135
- **Events, communities, or topics** may also be used as the top-level namespace when appropriate.
138-
Use discretion to determine whether a notebook is primarily about an artifact library,
139-
a concept, or an event.
136+
Use discretion to determine if the narrative is primarily about an artifact library, a concept, or an event.
140137
- Namespaces must consist of more than one segment.
141138

142139
#### Metadata and Navigation
@@ -172,17 +169,17 @@ Differentiation between posts, pages, and presentations is by `type` metadata (a
172169

173170
### File system organization
174171

175-
| Directory | Description |
176-
|-----------|----------------------------------------------------------------------|
177-
| `src` | Source root for notebooks (Clojure and Markdown), images, data files |
178-
| `site` | Static assets of the Quarto website |
172+
| Directory | Description |
173+
|-----------|--------------------------------------------------------------|
174+
| `src` | Source root for namespaces, markdown, images, and data files |
175+
| `site` | Static assets of the Quarto website |
179176

180177
Non-Clojure files in `src` will be synced to `site`.
181178
Shared images can go in `src/images`,
182179
but prefer placing images and data files as siblings to your namespace under `src`.
183180
All files in `src` should go under a subdirectory,
184181
so that it is clear they are not part of the static configuration of `site`.
185-
Clojure namespaces are built to markdown files under `site/{my/namespaced/notebook.qmd}`.
182+
Clojure namespaces are built to markdown files under `site/{my/namespaced/awesome-idea.qmd}`.
186183
Subdirectories of `site` are git ignored and considered temporary build artifacts, safe to clean up.
187184
Quarto builds all the markdown into HTML in `_site` for preview and deploy.
188185
While developing, Clay uses `temp` to build and serve HTML files.
@@ -203,7 +200,7 @@ Goal: Constellations, not cabinets.
203200

204201
### Dependency management
205202

206-
A single `deps.edn` file is shared across all notebooks.
203+
A single `deps.edn` file is shared.
207204

208205
Pros:
209206

@@ -214,7 +211,7 @@ Cons:
214211

215212
* Version conflicts must be manually resolved.
216213
* Only one version per dependency.
217-
* Notebooks aren’t self-contained.
214+
* Pages and posts aren’t self-contained.
218215

219216
Future:
220217

deps.edn

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,8 @@
1919
:aliases
2020
{;; Build the site with `clojure -M:clay -a [:markdown]`
2121
;; Run Clay in watch mode with `clojure -M:clay`
22-
:clay {:main-opts ["-m" "scicloj.clay.v2.main"]}
22+
:clay {:main-opts ["-m" "scicloj.clay.v2.main"]
23+
:jvm-opts ["-Dclojure.main.report=stderr"]}
2324
;; When debugging libraries
2425
:local {:override-deps {org.scicloj/clay {:local/root "../clay"}
2526
org.scicloj/kindly-advice {:local/root "../kindly-advice"}}}

0 commit comments

Comments
 (0)