Skip to content

Commit d80a154

Browse files
committed
Changes following review comments.
Signed-off-by: davem-bis <68955845+davem-bis@users.noreply.github.com>
1 parent f94a5ea commit d80a154

2 files changed

Lines changed: 12 additions & 4 deletions

File tree

.devcontainer/post-create-command.sh

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,13 @@
22

33
# This script is intended to be run by an Ubuntu dev container
44

5+
# Exit immediately if any command returns a non-zero code
6+
set -e
7+
58
# Install OS-level dependencies
9+
10+
# Note that msodbcsql18 install requires the Microsoft APT repo to be available to this script. If this script is used
11+
# in conjunction with a container image from the Microsoft container registry this should already be the case.
612
ALL_DEPENDENCIES="libpq-dev netcat-traditional unixodbc-dev default-jdk msodbcsql18"
713

814
sudo apt-get clean && sudo apt-get -y update && sudo ACCEPT_EULA='Y' apt-get -y install $ALL_DEPENDENCIES

docs/development.md

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,15 +15,17 @@ Before you begin, ensure you have the following installed on your machine. Exact
1515

1616
The development environment of SQLMesh depends both on:
1717

18-
* Python functionality (e.g. `SIGUSR1`) that is only available on UNIX systems, and;
19-
* Symbolic links in the repository which, whilst available on Windows, typically require additional permissions for the process running git.
18+
* Symbolic links in the repository which, whilst available on Windows, typically require additional permissions for the process running git, and;
19+
* Some Python functionality (e.g. `SIGUSR1`) that is only available on UNIX systems. Whilst this functionality is gated so shouldn't error on Windows, the development container enables its use.
2020

2121
For the Python functionality, a dev container is provided to develop against Ubuntu 24 with Python 3.12.
2222

2323
For symbolic links, you must ensure that when checking out the repository:
2424

25-
- The git configuration `core.symlinks` is set to `true`
26-
- The process that git runs as is permitted to create symbolic links. This can typically be done by running git as an administrator, or enabling [developer mode on Windows](https://learn.microsoft.com/en-us/windows/advanced-settings/developer-mode).
25+
* The git configuration `core.symlinks` is set to `true` (this also needs to be done before bind mount, i.e. when the dev container is started)
26+
* The process that git runs as is permitted to create symbolic links. This can typically be done by running git as an administrator, or enabling [developer mode on Windows](https://learn.microsoft.com/en-us/windows/advanced-settings/developer-mode).
27+
28+
Dev containers are supported by [a number of IDEs](https://containers.dev/supporting.html). For developers using VSCode, [Microsoft has a tutorial on how to use dev containers](https://code.visualstudio.com/docs/devcontainers/tutorial).
2729

2830
## Virtual environment setup
2931

0 commit comments

Comments
 (0)