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

Update readme #6011

Merged
merged 2 commits into from
Jul 17, 2018
Merged

Update readme #6011

merged 2 commits into from
Jul 17, 2018

Conversation

fjy
Copy link
Contributor

@fjy fjy commented Jul 17, 2018

No description provided.

@gianm
Copy link
Contributor

gianm commented Jul 17, 2018

@fjy could you include the incubation disclaimer too (https://incubator.apache.org/guides/branding.html#disclaimers)? Perhaps also some of the same explanatory language on https://druid.apache.org/.

The disclaimer is:

Disclaimer: Apache Druid is an effort undergoing incubation at The Apache Software Foundation (ASF), sponsored by the Apache Incubator. Incubation is required of all newly accepted projects until a further review indicates that the infrastructure, communications, and decision making process have stabilized in a manner consistent with other successful ASF projects. While incubation status is not necessarily a reflection of the completeness or stability of the code, it does indicate that the project has yet to be fully endorsed by the ASF.

@fjy
Copy link
Contributor Author

fjy commented Jul 17, 2018

@gianm added disclaimer

Copy link
Contributor

@gianm gianm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Would you like to include any of the other language from https://druid.apache.org/?

@fjy
Copy link
Contributor Author

fjy commented Jul 17, 2018

@gianm I think we should try to direct people to the Druid webpage as much as possible and avoid repeating information in too many places.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants