-
Notifications
You must be signed in to change notification settings - Fork 521
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
Workflow #275
base: master
Are you sure you want to change the base?
Workflow #275
Changes from 7 commits
82afdbe
ac01fa3
25f1745
bdc884a
f35c9ab
b024c3c
345ae0d
e1895b9
5ead5ef
967a3d6
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,35 @@ | ||
{ | ||
|
||
"env": { | ||
"browser": true, | ||
"es2021": true, | ||
"global": true | ||
}, | ||
"extends": "eslint:recommended", | ||
"overrides": [ | ||
{ | ||
"files": ["**/*.test.js"], | ||
"env": { "jest": true }, | ||
"plugins": ["jest"], | ||
"extends": ["plugin:jest/recommended"], | ||
"rules": { "jest/prefer-expect-assertions": "off" } | ||
}, | ||
{ | ||
"files": ["**/*.cy.js"], | ||
"env": { "cypress/globals": true }, | ||
"plugins": ["cypress"], | ||
"extends": ["plugin:cypress/recommended"], | ||
"rules": { | ||
"cypress/no-unnecessary-waiting": "off", | ||
"no-unused-vars": "off" | ||
} | ||
} | ||
|
||
], | ||
"parserOptions": { | ||
"ecmaVersion": "latest", | ||
"sourceType": "module" | ||
}, | ||
"rules": { | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,12 @@ | ||
|
||
added 52 packages, and audited 371 packages in 3s | ||
|
||
54 packages are looking for funding | ||
run `npm fund` for details | ||
|
||
3 high severity vulnerabilities | ||
|
||
To address all issues, run: | ||
npm audit fix | ||
|
||
Run `npm audit` for details. | ||
Comment on lines
+1
to
+12
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The changes indicate that there are 3 high severity vulnerabilities in your dependencies. It's recommended to address these issues as soon as possible to maintain the security of your application. You can do this by running + npm audit fix Also, consider setting up a process to regularly check for and resolve vulnerabilities in your dependencies. This could be a scheduled job that runs |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
When i ran a test on the login i got a failed test in wich said that localstorage was empty. |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
{ | ||
"presets": [["@babel/preset-env", { "targets": { "node": "current" } }]] | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
{} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,8 @@ | ||
import router from "./router/index.js"; | ||
import ui from "./ui/index.js"; | ||
|
||
ui() | ||
router() | ||
ui(); | ||
router(); | ||
Comment on lines
+5
to
+6
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Comment on lines
+5
to
+6
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There should be a newline at the end of the file. This is a common convention that makes it easier to concatenate files and work with them on the command line. Some tools might not work correctly with files that don't end with a newline. - \ No newline at end of file
+ - \ No newline at end of file
+ There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There should be a newline at the end of the file. This is a common convention that makes it easier to concatenate files and work with them on the command line. Some tools might not work correctly with files that don't end with a newline. - \ No newline at end of file
+ - \ No newline at end of file
+ |
||
// h | ||
console.log(dogcat) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CodeRabbit
The changes indicate that there are 3 high severity vulnerabilities in your dependencies. It's recommended to address these issues as soon as possible to maintain the security of your application. You can do this by running
npm audit fix
. If automatic fixing is not possible, you may need to manually update or replace the affected packages.+ npm audit fix
Also, consider setting up a process to regularly check for and resolve vulnerabilities in your dependencies. This could be a scheduled job that runs
npm audit
and alerts you if any vulnerabilities are found.