Skip to content

Commit

Permalink
Remove unnecessary execution of yarn build:production
Browse files Browse the repository at this point in the history
  • Loading branch information
mark-monteiro committed Mar 20, 2020
1 parent 6894602 commit 887e9c2
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 8 deletions.
2 changes: 1 addition & 1 deletion .ci/azure-pipelines-main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:
displayName: "Build Web Client"
condition: and(succeeded(), or(contains(variables['System.PullRequest.TargetBranch'], 'release'), contains(variables['System.PullRequest.TargetBranch'], 'master'), contains(variables['Build.SourceBranch'], 'release'), contains(variables['Build.SourceBranch'], 'master')), eq(variables['BuildConfiguration'], 'Release'), in(variables['Build.Reason'], 'PullRequest', 'IndividualCI', 'BatchedCI', 'BuildCompletion'))
inputs:
script: yarn install && yarn build:production
script: yarn install
workingDirectory: $(Agent.TempDirectory)/jellyfin-web

- task: CopyFiles@2
Expand Down
2 changes: 1 addition & 1 deletion .ci/azure-pipelines-windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ jobs:
displayName: "Build Web Client"
condition: and(succeeded(), or(contains(variables['System.PullRequest.TargetBranch'], 'release'), contains(variables['System.PullRequest.TargetBranch'], 'master'), contains(variables['Build.SourceBranch'], 'release'), contains(variables['Build.SourceBranch'], 'master')), in(variables['Build.Reason'], 'PullRequest', 'IndividualCI', 'BatchedCI', 'BuildCompletion'))
inputs:
script: yarn install && yarn build:production
script: yarn install
workingDirectory: $(Agent.TempDirectory)/jellyfin-web

- task: CopyFiles@2
Expand Down
3 changes: 1 addition & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@ ARG JELLYFIN_WEB_VERSION=master
RUN apk add curl git \
&& curl -L https://github.com/jellyfin/jellyfin-web/archive/${JELLYFIN_WEB_VERSION}.tar.gz | tar zxf - \
&& cd jellyfin-web-* \
&& yarn install \
&& yarn build:production \
&& yarn install
&& mv dist /dist

FROM mcr.microsoft.com/dotnet/core/sdk:${DOTNET_VERSION}-buster as builder
Expand Down
3 changes: 1 addition & 2 deletions Dockerfile.arm
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,7 @@ ARG JELLYFIN_WEB_VERSION=master
RUN apk add curl git \
&& curl -L https://github.com/jellyfin/jellyfin-web/archive/${JELLYFIN_WEB_VERSION}.tar.gz | tar zxf - \
&& cd jellyfin-web-* \
&& yarn install \
&& yarn build:production \
&& yarn install
&& mv dist /dist


Expand Down
3 changes: 1 addition & 2 deletions Dockerfile.arm64
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,7 @@ ARG JELLYFIN_WEB_VERSION=master
RUN apk add curl git \
&& curl -L https://github.com/jellyfin/jellyfin-web/archive/${JELLYFIN_WEB_VERSION}.tar.gz | tar zxf - \
&& cd jellyfin-web-* \
&& yarn install \
&& yarn build:production \
&& yarn install
&& mv dist /dist


Expand Down

0 comments on commit 887e9c2

Please sign in to comment.