-
Notifications
You must be signed in to change notification settings - Fork 65
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
Remove stray semicolon in CSS #507
Conversation
Please add a changelog entry and ensure you are in the AUTHORS file. Or, I could do a full pass on the CSS file myself and fix this in the process if you'd prefer. |
Oops. Forgot again. I will update. Thanks. |
@hildjj I broke up my PR into 3 different ones, so as to not base them off each other. You will have to merge the overlapping CHANGELOG. |
CHANGELOG.md
Outdated
@@ -11,6 +11,7 @@ Unreleased | |||
### New features | |||
|
|||
### Bug fixes | |||
- Remove stray semicolon in CSS |
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.
Please add a link to this PR. If you wouldn't mind fixing the one for EOI (#506) at the same time, I'd appreciate it.
For the overlapping changelog, I'm going to have you rebase each PR before merging. |
So after you approve all 3, I rebase them. Correct? |
yes. Let's finish the discussion of #509, then tell me what order you want me to apply them. |
Or if it's easier, just combine them into one PR. |
I think keep them separate.
|
No description provided.