-
-
Notifications
You must be signed in to change notification settings - Fork 4
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #412 from sasjs/issue-64
Improve adding and removing comments in SAS code
- Loading branch information
Showing
13 changed files
with
10,003 additions
and
15,291 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
## Issue | ||
|
||
Link any related issue(s) in this section. | ||
|
||
## Intent | ||
|
||
What this PR intends to achieve. | ||
|
||
## Implementation | ||
|
||
What code changes have been made to achieve the intent. | ||
|
||
## Checks | ||
|
||
- [ ] Code is formatted correctly (`npm run lint`). | ||
- [ ] All unit tests are passing (`npm run test:unit`). | ||
- [ ] Reviewer is assigned. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,33 @@ | ||
# This workflow will do a clean install of node dependencies, build the source code and run tests across different versions of node | ||
# For more information see: https://help.github.com/actions/language-and-framework-guides/using-nodejs-with-github-actions | ||
|
||
name: SASjs VS Code Extension Run Tests on Windows | ||
|
||
on: | ||
pull_request: | ||
|
||
jobs: | ||
test: | ||
runs-on: windows-latest | ||
|
||
strategy: | ||
matrix: | ||
node-version: [lts/hydrogen] | ||
|
||
steps: | ||
- uses: actions/checkout@v2 | ||
- name: Use Node.js ${{ matrix.node-version }} | ||
uses: actions/setup-node@v2 | ||
with: | ||
node-version: ${{ matrix.node-version }} | ||
|
||
- name: Install Dependencies | ||
run: npm ci | ||
|
||
- name: Run Unit Tests | ||
run: npm run test:unit | ||
|
||
- name: Build Package | ||
run: npm run vscode:prepublish | ||
env: | ||
CI: true |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,39 @@ | ||
# This workflow will do a clean install of node dependencies, build the source code and run tests across different versions of node | ||
# For more information see: https://help.github.com/actions/language-and-framework-guides/using-nodejs-with-github-actions | ||
|
||
name: SASjs VS Code Extension Run Tests | ||
|
||
on: | ||
pull_request: | ||
|
||
jobs: | ||
test: | ||
runs-on: ubuntu-latest | ||
|
||
strategy: | ||
matrix: | ||
node-version: [lts/hydrogen] | ||
|
||
steps: | ||
- uses: actions/checkout@v2 | ||
- name: Use Node.js ${{ matrix.node-version }} | ||
uses: actions/setup-node@v2 | ||
with: | ||
node-version: ${{ matrix.node-version }} | ||
|
||
- name: Check npm audit | ||
run: npm audit --production --audit-level=low | ||
|
||
- name: Install Dependencies | ||
run: npm ci | ||
|
||
- name: Check Code Style | ||
run: npm run lint | ||
|
||
- name: Run Unit Tests | ||
run: npm run test:unit | ||
|
||
- name: Build Package | ||
run: npm run vscode:prepublish | ||
env: | ||
CI: true |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,3 +4,4 @@ node_modules | |
.vscode-test/ | ||
*.vsix | ||
.DS_Store | ||
/coverage |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
module.exports = { | ||
preset: 'ts-jest/presets/js-with-ts', | ||
resetMocks: true, | ||
restoreMocks: true, | ||
testEnvironment: 'node', | ||
transform: { | ||
'^.+\\.ts?$': 'ts-jest' | ||
}, | ||
// INFO: uncomment coverageThreshold when required coverage is achieved | ||
// coverageThreshold: { | ||
// global: { | ||
// branches: 80, | ||
// functions: 80, | ||
// lines: 80, | ||
// statements: -10 | ||
// } | ||
// }, | ||
collectCoverageFrom: ['src/**/{!(index),}.ts'], | ||
testMatch: ['**/*spec.[j|t]s?(x)'] | ||
} |
Oops, something went wrong.