Skip to content

Enable and fix latest pedantic lints #202

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jan 29, 2020
Merged

Enable and fix latest pedantic lints #202

merged 1 commit into from
Jan 29, 2020

Conversation

kevmoo
Copy link
Collaborator

@kevmoo kevmoo commented Jan 28, 2020

No description provided.

@kevmoo kevmoo requested a review from robbecker-wf January 28, 2020 22:55
@kevmoo
Copy link
Collaborator Author

kevmoo commented Jan 28, 2020

Zero clue how to get this working with abide – let me know if I should do something else...

@robbecker-wf
Copy link
Contributor

Thanks! I'll take a look.

robrbecker
robrbecker approved these changes Jan 29, 2020
View reviewed changes
Copy link
Member

@robrbecker robrbecker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. We should probably just move to use pedantic directly huh?

@robrbecker robrbecker merged commit a13132d into SpinlockLabs:master Jan 29, 2020
@kevmoo
Copy link
Collaborator Author

kevmoo commented Jan 29, 2020

Looks good. We should probably just move to use pedantic directly huh?

Yup!

@kevmoo kevmoo deleted the lints branch January 29, 2020 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@robrbecker robrbecker robrbecker approved these changes

@robbecker-wf robbecker-wf Awaiting requested review from robbecker-wf

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kevmoo @robbecker-wf @robrbecker