Skip to content

Commit 54d006a

Browse files
authored
Use separate target for docker publishing (#21)
* Use separate target for docker publishing * Makefile optimizations
1 parent 24a6547 commit 54d006a

3 files changed

Lines changed: 12 additions & 5 deletions

File tree

.github/workflows/node.js.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,6 @@ jobs:
2727
uses: actions/setup-node@v1
2828
with:
2929
node-version: ${{ matrix.node-version }}
30-
- run: make
30+
- run: make github-ci
3131
env:
3232
TEST_CONFIGURATION_ACCESS_TOKEN: ${{ secrets.TEST_CONFIGURATION_ACCESS_TOKEN }}

Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,4 +10,4 @@ RUN npm ci
1010
RUN echo '//registry.npmjs.org/:_authToken=${NPM_TOKEN}' > .npmrc
1111

1212
ENTRYPOINT ["make"]
13-
CMD ["publish"]
13+
CMD ["publish-docker"]

Makefile

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
.PHONY: all
22
all: clean ci lint build cover
33

4+
.PHONY: github-ci
5+
github-ci: ci lint test
6+
47
.PHONY: init
58
init:
69
npm install
@@ -38,8 +41,8 @@ update:
3841
npm outdated
3942
npm install --package-lock-only
4043

41-
.PHONY: check_git
42-
check_git:
44+
.PHONY: check-git
45+
check-git:
4346
git fetch --depth 1 origin
4447
git diff origin/master --exit-code
4548

@@ -48,5 +51,9 @@ ci:
4851
npm ci || true
4952

5053
.PHONY: publish
51-
publish: clean ci test check_git
54+
publish: clean ci test check-git
55+
npm publish
56+
57+
.PHONY: publish-docker
58+
publish-docker: ci test
5259
npm publish

0 commit comments

Comments
 (0)