Skip to content

Commit 9b4f433

Browse files
committed
write doc comments
1 parent a47fcb4 commit 9b4f433

File tree

2 files changed

+24
-2
lines changed

2 files changed

+24
-2
lines changed

.submodules/__main__.py

+9-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,15 @@
22
© Ocado Group
33
Created on 02/03/2024 at 23:46:28(+00:00).
44
5-
TODO: write script description.
5+
This file is used to configure CFL's submodules using the global-config defined
6+
in the config.jsonc file in this directory.
7+
8+
By default, the global-config is *merged* into any existing config within each
9+
submodule. That is, any values defined in the global-config will override the
10+
values found in a submodule's config but if a submodule has key:value pairs not
11+
present in the global-config, they will remain. However, in some cases, the
12+
behavior is to override the values (values not present in the global-config will
13+
be removed).
614
"""
715

816
import json

.submodules/config.jsonc

+15-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,18 @@
1-
// TODO: Write comment
1+
// This file is used to define a global-config for the submodules within CFL's
2+
// workspace. This saves us having to manually repeat common configurations
3+
// across 2+ submodules and avoids human error when accidentally copy/pasting
4+
// something incorrectly.
5+
//
6+
// WARNING: This file is written in JSONC format (JSON with comments). However,
7+
// only single lines comments with no preceding text are allowed. For example:
8+
// - "// single-line comment, no preceding text" ✅
9+
// - "{"age": 27} // single-line comment, some preceding text" 🚫
10+
// - "/* multi-line comment, no preceding text */" 🚫
11+
// - "{"age": 27} /* multi-line comment, some preceding text */" 🚫
12+
//
13+
// This config is processed by the __main__.py file in this directory. To
14+
// understand more about this shape of the config data and how it's processed,
15+
// please read __main__.py.
216
{
317
"base": {
418
"description": "Base configuration to be inherited by all other configurations.",

0 commit comments

Comments
 (0)