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.

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

Release 0.5 #34

Closed
wants to merge 50 commits into from
Closed

Release 0.5 #34

wants to merge 50 commits into from

Conversation

Bigomby
Copy link
Contributor

@Bigomby Bigomby commented Aug 23, 2016

No description provided.

@Bigomby
Merge tag '0.4-beta3' into develop
a6a903f 
Version 0.4-beta3
@Bigomby
Refactor (#8)
983b9da 
Refactor the application to make it simplier.

The point is to have a minimal core and multiple reusable plugins to add functionality.

- No bidirectional references between structs.
- Remove logic that can be built into a plugin (limiter)
- Safe use of channels
- Use interfaces to hide implementation
- Loosely coupled modules
  - Reporter
  - Pipeline 

Closes #6
@Bigomby
Update README.md
809055a
@Bigomby
✨ Implemented multiple reports per message (#15)
32af15d 
Closes #11
@Bigomby
✨ Remove PushData() and add PopOpts() methods (#17)
d86acde 
Closes #16
@Bigomby
✨ Make rbforwarder.Report an interface
cbbb3ff
@Bigomby
Merge pull request #20 from redBorder/feature/reporter-interface
30d3de1 
Make rbforwarder.Report an interface
@Bigomby
✨ Add batcher component
fdca418 
Closes #10
@Bigomby
Merge pull request #18 from redBorder/feature/batcher-component
c1c9195 
Add batcher component
@Bigomby
✅ Add message.go testing
98c582e
@Bigomby
Merge pull request #24 from redBorder/feature/message-test
bce7d64 
Add message.go testing
@Bigomby
💄 Fix wrong test name
a90f28b
@Bigomby
✨ Replaced message interface for a struct
3d36c89 
This decrease the risk of not pass the original info to the next component on the pipeline
@Bigomby
✨ Add opaque to produce function
3990371 
It can be recovered on reports
@Bigomby
✨ Make message payload private
b2de50a 
It is now exposed via a method. Assertions are not necessary anymore and
there is error handling.
@Bigomby
Merge pull request #26 from redBorder/feature/message-struct
acef863 
Replaced message interface for a struct
@Bigomby
🐛 Fix missing initialization of map
19f84a5
@Bigomby
✨ Add http sender component
544cacf 
Closes #3
@Bigomby
Merge pull request #27 from redBorder/feature/http-component
fdcb8cc 
Add http sender component
@Bigomby
💄 Rename types -> utils
989a419
@Bigomby
✨ Add limiter component
522441b
@Bigomby
Merge pull request #28 from redBorder/feature/limiter
f704bdd 
Add limiter component
@Bigomby
🐛 New batcher implementation
730bd05 
Fixes #29
@Bigomby
Merge pull request #30 from redBorder/feature/fix-batcher
a4f7cb6 
New batcher implementation
@Bigomby
✨ Add new example using HTTP and batch
70c9a8d
@Bigomby
🐛 Make rbforwarder.Report public
9cb42c0 
It was impossible to obtain a report for the user
@Bigomby
✨ Improve example
87779e4
@Bigomby
✨ Make http client public
9f9121c
@Bigomby
🐎 Add benchmarks
7a0329d
@Bigomby
Merge pull request #31 from redBorder/feature/example
708c6fb 
Add new example using HTTP and batch
@Bigomby Bigomby self-assigned this Aug 23, 2016
@coveralls
Copy link

coveralls commented Aug 24, 2016 •
edited
Loading

Coverage Status

Coverage increased (+43.8%) to 100.0% when pulling 6e6595d on release/0.5 into 1a8c423 on master.

@coveralls
Copy link

coveralls commented Aug 25, 2016 •
edited
Loading

Coverage Status

Coverage increased (+42.4%) to 98.675% when pulling 031ae81 on release/0.5 into 1a8c423 on master.

@Bigomby
🐛 Fix limiter component
695f539 
It was using the old behavior
@coveralls
Copy link

Coverage Status

Coverage increased (+42.0%) to 98.238% when pulling 695f539 on release/0.5 into 1a8c423 on master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+42.0%) to 98.238% when pulling 695f539 on release/0.5 into 1a8c423 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+42.0%) to 98.238% when pulling 695f539 on release/0.5 into 1a8c423 on master.

@coveralls
Copy link

coveralls commented Aug 25, 2016 •
edited
Loading

Coverage Status

Coverage increased (+40.5%) to 96.767% when pulling 0f54b99 on release/0.5 into 1a8c423 on master.

@Bigomby Bigomby closed this Sep 6, 2016
@Bigomby Bigomby deleted the release/0.5 branch September 6, 2016 06:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Bigomby Bigomby

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Bigomby @coveralls