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

feature: logux actions behaive as action params #29

Merged
merged 1 commit into from Nov 8, 2018

Conversation

@dsalahutdinov
Copy link
Collaborator

@dsalahutdinov dsalahutdinov commented Oct 28, 2018

#8

@coveralls
Copy link

@coveralls coveralls commented Oct 28, 2018

Pull Request Test Coverage Report for Build 157

  • 6 of 6 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.005%) to 99.043%

Totals Coverage Status
Change from base Build 156: 0.005%
Covered Lines: 414
Relevant Lines: 418

💛 - Coveralls

@dsalahutdinov dsalahutdinov removed the WIP label Oct 29, 2018
key { 'name' }
value { 'test1' }
skip_create
intialize_wit do
Copy link
Member

@ai ai Oct 29, 2018

Описка?

Copy link
Collaborator Author

@dsalahutdinov dsalahutdinov Oct 29, 2018

да, эта фабрика похоже не используется, щас выпилим

@ai
Copy link
Member

@ai ai commented Nov 2, 2018

@dsalahutdinov тесты не проходят

@dsalahutdinov dsalahutdinov requested a review from wilddima Nov 4, 2018
@ai
Copy link
Member

@ai ai commented Nov 6, 2018

@wilddima принимаем PR?

Copy link
Member

@wilddima wilddima left a comment

👍

@dsalahutdinov dsalahutdinov merged commit 01689f4 into logux:master Nov 8, 2018
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants