Skip to content

Commit 3687e2b

Browse files
committed
Fix codacy issues
1 parent dea9fbe commit 3687e2b

File tree

4 files changed

+11
-10
lines changed

4 files changed

+11
-10
lines changed

.github/actions/build-environment/Dockerfile

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ RUN apt-get update --fix-missing \
5353
&& rm -rf /var/lib/apt/lists/*
5454

5555
# Add tini
56-
RUN wget --quiet https://github.com/krallin/tini/releases/download/v0.19.0/tini -O /tini && \
56+
RUN curl -fsSL https://github.com/krallin/tini/releases/download/v0.19.0/tini -o /tini && \
5757
chmod +x /tini
5858

5959
# Install Docker in Container
@@ -103,7 +103,7 @@ ENV PATH=$RESOURCES_PATH/.pyenv/shims:$RESOURCES_PATH/.pyenv/bin:$PATH \
103103

104104
# Install web development tools
105105
RUN apt-get update \
106-
&& curl -sL https://deb.nodesource.com/setup_12.x | bash - \
106+
&& curl -sL https://deb.nodesource.com/setup_14.x | bash - \
107107
&& apt-get install -y --no-install-recommends nodejs \
108108
&& npm install -g yarn@1 \
109109
# Clean up
@@ -123,10 +123,11 @@ RUN mkdir -p /github/home/.ssh/ \
123123

124124
# Settings and configurations
125125
ENV \
126-
# Flush log message immediately to stdout
127-
PYTHONUNBUFFERED=true \
128126
# Use local folder as pipenv virtualenv
129-
PIPENV_VENV_IN_PROJECT=true
127+
PIPENV_VENV_IN_PROJECT=true \
128+
# Flush log message immediately to stdout
129+
PYTHONUNBUFFERED=true
130+
130131

131132
# Install Universal-Build
132133
# hadolint ignore=DL3013

.github/actions/build-environment/entrypoint.sh

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,17 +19,17 @@ if [ -n "$GITHUB_TOKEN" ]; then
1919
fi
2020

2121
if [ -n "$INPUT_CONTAINER_REGISTRY_USERNAME" ] && [ -n "$INPUT_CONTAINER_REGISTRY_PASSWORD" ]; then
22-
docker login $INPUT_CONTAINER_REGISTRY_URL -u "$INPUT_CONTAINER_REGISTRY_USERNAME" -p "$INPUT_CONTAINER_REGISTRY_PASSWORD"
22+
docker login "$INPUT_CONTAINER_REGISTRY_URL" -u "$INPUT_CONTAINER_REGISTRY_USERNAME" -p "$INPUT_CONTAINER_REGISTRY_PASSWORD"
2323
BUILD_ARGS="$BUILD_ARGS --container-registry-url=$INPUT_CONTAINER_REGISTRY_URL"
2424
BUILD_ARGS="$BUILD_ARGS --container-registry-username=$INPUT_CONTAINER_REGISTRY_USERNAME"
2525
BUILD_ARGS="$BUILD_ARGS --container-registry-password=$INPUT_CONTAINER_REGISTRY_PASSWORD"
2626
fi
2727

2828
# Navigate to working directory, if provided
2929
if [ -n "$INPUT_WORKING_DIRECTORY" ]; then
30-
cd $INPUT_WORKING_DIRECTORY
30+
cd "$INPUT_WORKING_DIRECTORY"
3131
else
32-
cd $GITHUB_WORKSPACE
32+
cd "$GITHUB_WORKSPACE"
3333
fi
3434

3535
if [ -n "$INPUT_PYPI_TOKEN" ]; then
@@ -55,4 +55,4 @@ fi
5555
set -x
5656

5757
# Execute build script
58-
python -u build.py $BUILD_ARGS
58+
python -u build.py "$BUILD_ARGS"

build.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,6 @@ def _check(args: Dict[str, Union[bool, str]]):
5757

5858
def _make(args: Dict[str, Union[bool, str]]):
5959
"""Build the library."""
60-
6160
try:
6261
# Ensure there are no old builds
6362
rmtree(os.path.join(HERE, "dist"))

src/lazydocs/generation.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -966,5 +966,6 @@ def generate_docs(
966966

967967
# Write mkdocs pages file
968968
print("Writing mkdocs .pages file.")
969+
# TODO: generate navigation items to fix problem with naming
969970
with open(os.path.join(output_path, ".pages"), "w") as f:
970971
f.write(_MKDOCS_PAGES_TEMPLATE.format(overview_file=overview_file))

0 commit comments

Comments
 (0)