Skip to content
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

Format the repository and add a workflow for formatting and linting #1174

Merged
merged 4 commits into from
Jan 18, 2021

Conversation

arqunis
Copy link
Member

@arqunis arqunis commented Jan 7, 2021

Now that we've exhausted all of our pull requests (except for #878, not sure what to do with it), we can advance towards syntactical consistency. This entails formatting code with options defined in rustfmt.toml before merging. To ensure this, a new workflow called lint is added that will check for warnings and suggestions from clippy and formatting so code adheres to the output of rustfmt.

@arqunis arqunis added enhancement An improvement to Serenity. ci Related to the continuous integration. labels Jan 7, 2021
@arqunis arqunis requested a review from Lakelezz January 12, 2021 10:18
@arqunis arqunis merged commit 9bbb25a into serenity-rs:current Jan 18, 2021
@arqunis arqunis deleted the fmt branch January 18, 2021 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Related to the continuous integration. enhancement An improvement to Serenity.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants