-
Notifications
You must be signed in to change notification settings - Fork 28
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
Repo Clean Up (Removals) #317
Repo Clean Up (Removals) #317
Conversation
@pllim this addresses a couple of issues and the CI is passing. Would you be able to do a quick review? @Witchblade101 is out of town on vacation and I don't want this to go stale. |
Err... did you try a rebase?
|
Move this to "docs" requirement Line 21 in f49b529
|
I cannot tell. It has merge commits. |
I see the merge commits from |
I'll have to trust your judgement on this one since you are the maintainer after all. Thanks for checking! |
… gitignore file as well
08c1030
to
83ae047
Compare
@Witchblade101 this one is ready for review. |
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
addresses #310, #315, #311, #306