Skip to content

Commit c5d9703

Browse files
authored
Update pymarkdownlnt devcontainer feature and test scripts (#27)
1 parent 2283944 commit c5d9703

File tree

6 files changed

+234
-0
lines changed

6 files changed

+234
-0
lines changed

.github/workflows/test.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ jobs:
1818
- django-upgrade
1919
- pyadr
2020
- pytest
21+
- pymarkdownlnt
2122
- pyupgrade
2223
- rstcheck
2324
- sshpass
@@ -45,6 +46,7 @@ jobs:
4546
- django-upgrade
4647
- pyadr
4748
- pytest
49+
- pymarkdownlnt
4850
- pyupgrade
4951
- rstcheck
5052
- sshpass
Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
{
2+
"id": "pymarkdownlnt",
3+
"version": "1.0.0",
4+
"name": "pymarkdownlnt (via pipx)",
5+
"documentationURL": "http://github.com/hspaans/devcontainer-features/tree/master/src/pymarkdownlnt",
6+
"licenseURL": "http://github.com/hspaans/devcontainer-features/tree/master/LICENSE",
7+
"description": "PyMarkdown is primarily a GitHub Flavored Markdown linter.",
8+
"options": {
9+
"version": {
10+
"default": "latest",
11+
"description": "Select the version to install.",
12+
"proposals": [
13+
"latest"
14+
],
15+
"type": "string"
16+
}
17+
},
18+
"installsAfter": [
19+
"ghcr.io/devcontainers-contrib/features/pipx-package",
20+
"ghcr.io/devcontainers/features/python",
21+
"ghcr.io/devcontainers/features/git"
22+
]
23+
}

src/pymarkdownlnt/install.sh

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
2+
set -e
3+
4+
. ./library_scripts.sh
5+
6+
# nanolayer is a cli utility which keeps container layers as small as possible
7+
# source code: https://github.com/devcontainers-contrib/nanolayer
8+
# `ensure_nanolayer` is a bash function that will find any existing nanolayer installations,
9+
# and if missing - will download a temporary copy that automatically get deleted at the end
10+
# of the script
11+
ensure_nanolayer nanolayer_location "v0.5.0"
12+
13+
14+
$nanolayer_location \
15+
install \
16+
devcontainer-feature \
17+
"ghcr.io/devcontainers-contrib/features/pipx-package:1.1.7" \
18+
--option package='pymarkdownlnt' --option version="$VERSION"
19+
20+
21+
22+
echo 'Done!'

src/pymarkdownlnt/library_scripts.sh

Lines changed: 170 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,170 @@
1+
2+
3+
clean_download() {
4+
# The purpose of this function is to download a file with minimal impact on container layer size
5+
# this means if no valid downloader is found (curl or wget) then we install a downloader (currently wget) in a
6+
# temporary manner, and making sure to
7+
# 1. uninstall the downloader at the return of the function
8+
# 2. revert back any changes to the package installer database/cache (for example apt-get lists)
9+
# The above steps will minimize the leftovers being created while installing the downloader
10+
# Supported distros:
11+
# debian/ubuntu/alpine
12+
13+
url=$1
14+
output_location=$2
15+
tempdir=$(mktemp -d)
16+
downloader_installed=""
17+
18+
_apt_get_install() {
19+
tempdir=$1
20+
21+
# copy current state of apt list - in order to revert back later (minimize contianer layer size)
22+
cp -p -R /var/lib/apt/lists $tempdir
23+
apt-get update -y
24+
apt-get -y install --no-install-recommends wget ca-certificates
25+
}
26+
27+
_apt_get_cleanup() {
28+
tempdir=$1
29+
30+
echo "removing wget"
31+
apt-get -y purge wget --auto-remove
32+
33+
echo "revert back apt lists"
34+
rm -rf /var/lib/apt/lists/*
35+
rm -r /var/lib/apt/lists && mv $tempdir/lists /var/lib/apt/lists
36+
}
37+
38+
_apk_install() {
39+
tempdir=$1
40+
# copy current state of apk cache - in order to revert back later (minimize contianer layer size)
41+
cp -p -R /var/cache/apk $tempdir
42+
43+
apk add --no-cache wget
44+
}
45+
46+
_apk_cleanup() {
47+
tempdir=$1
48+
49+
echo "removing wget"
50+
apk del wget
51+
}
52+
# try to use either wget or curl if one of them already installer
53+
if type curl >/dev/null 2>&1; then
54+
downloader=curl
55+
elif type wget >/dev/null 2>&1; then
56+
downloader=wget
57+
else
58+
downloader=""
59+
fi
60+
61+
# in case none of them is installed, install wget temporarly
62+
if [ -z $downloader ] ; then
63+
if [ -x "/usr/bin/apt-get" ] ; then
64+
_apt_get_install $tempdir
65+
elif [ -x "/sbin/apk" ] ; then
66+
_apk_install $tempdir
67+
else
68+
echo "distro not supported"
69+
exit 1
70+
fi
71+
downloader="wget"
72+
downloader_installed="true"
73+
fi
74+
75+
if [ $downloader = "wget" ] ; then
76+
wget -q $url -O $output_location
77+
else
78+
curl -sfL $url -o $output_location
79+
fi
80+
81+
# NOTE: the cleanup procedure was not implemented using `trap X RETURN` only because
82+
# alpine lack bash, and RETURN is not a valid signal under sh shell
83+
if ! [ -z $downloader_installed ] ; then
84+
if [ -x "/usr/bin/apt-get" ] ; then
85+
_apt_get_cleanup $tempdir
86+
elif [ -x "/sbin/apk" ] ; then
87+
_apk_cleanup $tempdir
88+
else
89+
echo "distro not supported"
90+
exit 1
91+
fi
92+
fi
93+
94+
}
95+
96+
97+
ensure_nanolayer() {
98+
# Ensure existance of the nanolayer cli program
99+
local variable_name=$1
100+
101+
local required_version=$2
102+
103+
local __nanolayer_location=""
104+
105+
# If possible - try to use an already installed nanolayer
106+
if [ -z "${NANOLAYER_FORCE_CLI_INSTALLATION}" ]; then
107+
if [ -z "${NANOLAYER_CLI_LOCATION}" ]; then
108+
if type nanolayer >/dev/null 2>&1; then
109+
echo "Found a pre-existing nanolayer in PATH"
110+
__nanolayer_location=nanolayer
111+
fi
112+
elif [ -f "${NANOLAYER_CLI_LOCATION}" ] && [ -x "${NANOLAYER_CLI_LOCATION}" ] ; then
113+
__nanolayer_location=${NANOLAYER_CLI_LOCATION}
114+
echo "Found a pre-existing nanolayer which were given in env variable: $__nanolayer_location"
115+
fi
116+
117+
# make sure its of the required version
118+
if ! [ -z "${__nanolayer_location}" ]; then
119+
local current_version
120+
current_version=$($__nanolayer_location --version)
121+
122+
123+
if ! [ $current_version == $required_version ]; then
124+
echo "skipping usage of pre-existing nanolayer. (required version $required_version does not match existing version $current_version)"
125+
__nanolayer_location=""
126+
fi
127+
fi
128+
129+
fi
130+
131+
# If not previuse installation found, download it temporarly and delete at the end of the script
132+
if [ -z "${__nanolayer_location}" ]; then
133+
134+
if [ "$(uname -sm)" = 'Linux x86_64' ] || [ "$(uname -sm)" = "Linux aarch64" ]; then
135+
tmp_dir=$(mktemp -d -t nanolayer-XXXXXXXXXX)
136+
137+
clean_up () {
138+
ARG=$?
139+
rm -rf $tmp_dir
140+
exit $ARG
141+
}
142+
trap clean_up EXIT
143+
144+
145+
if [ -x "/sbin/apk" ] ; then
146+
clib_type=musl
147+
else
148+
clib_type=gnu
149+
fi
150+
151+
tar_filename=nanolayer-"$(uname -m)"-unknown-linux-$clib_type.tgz
152+
153+
# clean download will minimize leftover in case a downloaderlike wget or curl need to be installed
154+
clean_download https://github.com/devcontainers-contrib/cli/releases/download/$required_version/$tar_filename $tmp_dir/$tar_filename
155+
156+
tar xfzv $tmp_dir/$tar_filename -C "$tmp_dir"
157+
chmod a+x $tmp_dir/nanolayer
158+
__nanolayer_location=$tmp_dir/nanolayer
159+
160+
161+
else
162+
echo "No binaries compiled for non-x86-linux architectures yet: $(uname -m)"
163+
exit 1
164+
fi
165+
fi
166+
167+
# Expose outside the resolved location
168+
export ${variable_name}=$__nanolayer_location
169+
170+
}

test/pymarkdownlnt/scenarios.json

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
{
2+
"test": {
3+
"image": "mcr.microsoft.com/devcontainers/base:ubuntu",
4+
"features": {
5+
"pymarkdownlnt": {}
6+
}
7+
}
8+
}

test/pymarkdownlnt/test.sh

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
#!/bin/bash -i
2+
3+
set -e
4+
5+
source dev-container-features-test-lib
6+
7+
check "pymarkdownlnt version" pymarkdownlnt version
8+
9+
reportResults

0 commit comments

Comments
 (0)