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

Enable analyzers, editorconfigs, and API docs #66

Merged
merged 18 commits into from
Apr 11, 2019

Conversation

Kieranties
Copy link
Owner

Description

Introduces a number of analyzers, small refactorings, and API docs for the public surface.
Also adds editorconfig settings to begin applying a consistent code style.

The work does contain breaking changes due to the renaming of types. This is okay while still in a 0.x release

Fixes #38

Change Type

  • Documentation: Changes to docs are included
  • Infrastructure: Changes to build scripts/non-deliverables
  • Bug Fix: Fixes an issue in implementation
  • New Feature: Adds new functionality in a non-breaking manner
  • Breaking Change: Implements a fix or feature in a breaking manner

Quality

  • Unit tests have been added/updated
  • Local builds/testing are successful
  • Code coverage has not decreased
  • Code comments have been provided (where appropriate)
  • Core styles have been followed
  • Documentation has been added/updated

@Kieranties Kieranties merged commit b0aed53 into master Apr 11, 2019
@Kieranties Kieranties added this to the 0.2.0 milestone May 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enable editorconfig and static analysis
1 participant