-
Notifications
You must be signed in to change notification settings - Fork 432
Page runs #4657
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
base: vue3-development
Are you sure you want to change the base?
Page runs #4657
Conversation
overrideConfigFile: './.eslintrc.js' | ||
Buffer: [ 'buffer', 'Buffer' ], | ||
}), | ||
// new ESLintPlugin({ |
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.
Try not to leave commented out code in the final code. If needed, uncomment it, if not then remove it.
port: 8080, | ||
hot: true, | ||
historyApiFallback: { | ||
// If the URL contains a product endpoint and we server a static file |
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.
This comment might be informative in the future. You could keep it.
width="400" | ||
> | ||
<template v-slot:activator="{ on }"> | ||
<template #activator="{ props }"> |
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.
Is it a best practice to replace v-slot with a hash mark?
this.resetDateTimes(); | ||
} | ||
}, | ||
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.
Deleting every single empty line will not make the code better but makes readability worse. Don't remove all of it but decide with the question: "Will it make the code readable?".
|
||
class CodeCheckerService extends BaseService { | ||
constructor() { | ||
super("CodeCheckerService", ServiceClient,"v6.61", true); |
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.
Didn't you remove the appended 'v' from an other place too? If I'm right, by removing this too, it will get rid of the 'v' in the whole version number.
}, | ||
mixins: [ VersionMixin ], | ||
data() { |
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.
What happened to the identation?
Migrated runs page