Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#28 user-register #41

Merged
merged 1 commit into from
May 6, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion .eslintrc.json
Original file line number Diff line number Diff line change
Expand Up @@ -9,5 +9,6 @@
"rules": {
"no-console": "warn",
"quotes": ["error", "single"]
}
},
"ignorePatterns": ["dist/**/*"]
}
44 changes: 29 additions & 15 deletions .github/workflows/workflow_for_ecomm.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,31 +8,45 @@ on:

jobs:
build:
#git hub offer virtual machines to run workflows so we will be using ubuntu lastest version its standand one
#we using ubuntu because we want to use lunex terminal
runs-on: ubuntu-latest

strategy:
matrix:
#each version we specify we be tested on
#we are only limited to 3 versions
node-version: [16.x, 18.x, 20.x]

node-version: [20.x]

steps:
- uses: actions/checkout@v3
- name: Use Node.js ${{ matrix.node-version }} #this will help use know which version we are testing on.
- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v3
with:
node-version: ${{ matrix.node-version }}
cache: 'npm' #this we help speed up the workflow by reusing dependencies from previous runs
- run: npm ci #run dependencies installing we using ci in place of 'npm install' becouse ci(clearn install) is smooth and faster
- run: npm run test --if-present #this line will run test script
- run: npm run lint --if-present # run tle
- run: npm run build --if-present # we using if statement because initial the code have no build script in packege.json
- run: npm run test:ci --if-present # this will run test with coverage flag

cache: 'npm'
- run: npm ci
- run: npm run test --if-present
- run: npm run build --if-present
- run: npm run test:ci --if-present
- run: npm run lint --if-present
- name: Upload coverage to Coveralls
uses: coverallsapp/github-action@v2
with:
allow-empty: true
allow-empty: true
env:
DB_PORT_DEV: ${{ secrets.DB_PORT_DEV }}
DB_USER_DEV: ${{ secrets.DB_USER_DEV }}
DB_PASSWORD_DEV: ${{ secrets.DB_PASSWORD_DEV }}
DB_NAME_DEV: ${{ secrets.DB_NAME_DEV }}
DB_HOST_DEV: ${{ secrets.DB_HOST_DEV }}
APP_URL: ${{ secrets.APP_URL }}
ALL: ${{ secrets.ALL }}
DOCS: ${{ secrets.DOCS }}
APP_PORT: ${{ secrets.APP_PORT }}
JWT_SECRET: ${{ secrets.JWT_SECRET }}
DB_PORT_TEST: ${{ secrets.DB_PORT_TEST }}
DB_USER_TEST: ${{ secrets.DB_USER_TEST }}
DB_PASSWORD_TEST: ${{ secrets.DB_PASSWORD_TEST }}
DB_NAME_TEST: ${{ secrets.DB_NAME_TEST }}
DB_HOST_TEST: ${{ secrets.DB_HOST_TEST }}
MAILERSEND_TOKEN: ${{ secrets.MAILERSEND_TOKEN }}
MAILERSEND_DOMAIN: ${{ secrets.MAILERSEND_DOMAIN }}


1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,4 @@ node_modules/
dist/
src/output.log
coverage/

Loading
Loading
You are viewing a condensed version of this merge commit. You can view the full changes here.