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

[#1232] update readme #1248

Merged
merged 16 commits into from May 8, 2019

Conversation

Projects
None yet
3 participants
@boggledekoog
Copy link
Contributor

commented May 2, 2019

[#1232] updated readme to contain all operators and modules. Also added link to the wiki and xml syntax highlighting

The list of operators is quite long now, we could consider moving it into the wiki.

@ChrizZz110
Copy link
Contributor

left a comment

Good job. Please have a look at the TM and ® symbols. Correct them in the whole readme.
Apache gets a ®
Apache Flink gets nothing
Flink gets a ®
HBase gets a ™
Accumulo gets a ®

Show resolved Hide resolved README.md Outdated
Show resolved Hide resolved README.md Outdated
Show resolved Hide resolved README.md Outdated
Show resolved Hide resolved README.md Outdated
Show resolved Hide resolved README.md Outdated
Show resolved Hide resolved README.md Outdated
Show resolved Hide resolved README.md Outdated
Show resolved Hide resolved README.md Outdated

@ChrizZz110 ChrizZz110 changed the title #1232 update readme [#1232] update readme May 3, 2019

@boggledekoog

This comment has been minimized.

Copy link
Contributor Author

commented May 7, 2019

The ticket can be reviewed again, I integrated all proposed changes.

I moved the list of operators from the README to the Wiki and added links to further operator documentation.
Also the Split operator was missing from the list.

@ChrizZz110
Copy link
Contributor

left a comment

One small change - then it is OK for me.

Show resolved Hide resolved README.md Outdated
Update README.md
Co-Authored-By: boggledekoog <juliangoetz632@gmail.com>
@ChrizZz110
Copy link
Contributor

left a comment

LGTM

@galpha galpha merged commit 9e23b18 into dbs-leipzig:develop May 8, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.