-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat(course): Add user role badge #119
Conversation
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.
We appreciate your interest in contributing to our project. As this is your first pull request, please ensure you have followed our Contributing Guidelines, as well as our Git/GitHub workflows. Adhering to these practices helps maintain our project's quality and ease of collaboration. If you're unsure about anything or need some help, don't hesitate to ask. Happy coding!
.eslintrc.json
Outdated
@@ -8,6 +8,6 @@ | |||
}, | |||
"ignorePatterns": ["dist/", "node_modules/"], | |||
"rules": { | |||
"@typescript-eslint/no-explicit-any": "warn" | |||
"@typescript-eslint/no-explicit-any": "off" |
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.
leave it warn
, use git commit ... --no-verify
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.
looks good 👍🏻
Types of changes
Checklist:
Description
I added a user role badge to indicate whether an account belongs to an admin, a regular user, or is not verified.