Skip to content

Commit

Permalink
Merge pull request #1149 from dondi/#1147-include-npm-lin
Browse files Browse the repository at this point in the history
#1147 include npm lint
  • Loading branch information
dondi authored Dec 4, 2024
2 parents 4c5bd50 + ebac79b commit 079811e
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions .github/workflows/node.js.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ jobs:
cache: 'npm'

- run: npm ci
- run: npm run lint
- run: npm run build --if-present
- run: npm test

Expand Down
4 changes: 2 additions & 2 deletions web-client/public/js/setup-handlers.js
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ export const setupHandlers = grnState => {

const explicitlySetStyle = element => {
const cssStyleDeclarationComputed = window.getComputedStyle(element);
const computedStyleObj = {};
const computedStyleObj = {};


for (let i = 0; i < cssStyleDeclarationComputed.length; i++) {
Expand All @@ -134,7 +134,7 @@ export const setupHandlers = grnState => {
}

if (computedStyleObj) {
Object.assign(element.style, computedStyleObj)
Object.assign(element.style, computedStyleObj);
}
};

Expand Down

0 comments on commit 079811e

Please sign in to comment.