Open
Conversation
Merge branch 'expr' into noschema
Feature decision table parser
* updated example to use store config file * fixing excelise v2 upgrade * update decisiontable example to work with xlsx file and fix go tests (#82) * update decisiontable example to work with xlsx file and fix go tests * removing duplicated flogo.json from test folder * updated decision table example readme * stabilising mod and sum files
Feature decision table
* Adding travis file with basic configuration Test travis file currently runs go tests and this will be enhanced further * travis file enhancements * updating travis file install latest flogo cli * restructuring travis file * resolving build failure issue with gomodules * fix build failures * pass correct env values to travis file
Merge from master
fixes issue - #91
* Add examples * Updating examples * Updating comments
* dtable code refactoring to support api model * adding dtable ruleapi model example * adding dtable interface * example modified * decision table api model refactored and example modified * Adding Readme * Update Readme * go tests added * code refactoring Co-authored-by: ykalidin <ykalidin@tibco.com>
* Updateing examples * Update Test
* Updating Readme * Update Readme * Updating Readme
rameshpolishetti
requested changes
Apr 29, 2020
Contributor
rameshpolishetti
left a comment
rameshpolishetti
left a comment
There was a problem hiding this comment.
Since this PR contains several features. It's not a good idea to merge at once, instead break this PR into feature-based PRs and submit one after another.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds following functionality: