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

[RFC] Move events to their correct event class as inner classes #233

Closed
wants to merge 8 commits into from

Conversation

ExE-Boss
Copy link
Member

@ExE-Boss ExE-Boss commented Jan 6, 2017

This PR does some work on #192.

Depends on:

To do list:

  • Consider moving other events as well (Leave that for another PR)

Completed:

  • Move ItemRegistrationEvent to ItemEvent as ItemEvent.Register
  • Move ItemIDNotFoundEvent to ItemEvent as ItemEvent.IDNotFound

@codecov-io
Copy link

codecov-io commented Jan 6, 2017

Current coverage is 13.76% (diff: 33.33%)

Merging #233 into master will increase coverage by <.01%

@@             master       #233   diff @@
==========================================
  Files           387        387          
  Lines         10839      10842     +3   
  Methods           0          0          
  Messages          0          0          
  Branches       1547       1547          
==========================================
+ Hits           1491       1492     +1   
- Misses         9271       9273     +2   
  Partials         77         77          

Powered by Codecov. Last update 70a4f22...6d7327e

@ExE-Boss ExE-Boss added breaking change This is going to break stuff ready This is ready to be worked on labels Jan 7, 2017
@ExE-Boss ExE-Boss self-assigned this Jan 7, 2017
@ExE-Boss ExE-Boss added the enhancement Nice to have features label Jan 7, 2017
@ExE-Boss ExE-Boss added this to the v0.1.0 milestone Jan 9, 2017
@RX14
Copy link
Contributor

RX14 commented Jan 9, 2017

Cleaned up and merged into master

@RX14 RX14 closed this Jan 9, 2017
@RX14 RX14 removed the ready This is ready to be worked on label Jan 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change This is going to break stuff enhancement Nice to have features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants