Streamline the Contribute section a bit #438
Conversation
kennknowles
force-pushed
the
contribution-guide
branch
from
77de036 to
a4f2e5c
Compare
| A great way to contribute is to join an existing effort. There are many | ||
| works in progress, some on branches because they are very incomplete. | ||
|
|
||
| ### Portability Framework |
There was a problem hiding this comment.
@herohde @bsidhom This is a fairly abbreviated link, but I thought it was the most important work-in-progress to highlight. It might be nice to break it out into efforts for each SDK and runner. Or maybe that should be just part of the linked doc. Anyhow, take a look at http://apache-beam-website-pull-requests.storage.googleapis.com/438/contribute/index.html#works-in-progress and see what you think.
There was a problem hiding this comment.
If at all, you could reference BEAM-2889 under a new portable Flink runner section. I'm not sure how much sense it makes to add directly to the linked doc because that already contains a reference to the umbrella "wordcount runs everywhere" bug (BEAM-2896).
There was a problem hiding this comment.
I think mentioning the Flink runner more prominently both here and on the portability page would be great. I also wanted to reference the progress sheet under status, so it might be a good context to call out Flink.
I don't think we should make it a new section, because it's not a new runner in the same way the others are. But I don't feel very strongly about it.
There was a problem hiding this comment.
Could I prevail upon one of you two to follow up this PR or issue a PR to my PR?
|
retest this please |
|
Ah, the test is failure because of Google's choice to eliminate "research" and only have "AI". A bit of an awkward moniker for FlumeJava. |
kennknowles
force-pushed
the
contribution-guide
branch
from
d264be5 to
b0c16fd
Compare
kennknowles
force-pushed
the
contribution-guide
branch
from
b0c16fd to
4284b6a
Compare
|
retest this please |
2 similar comments
|
retest this please |
|
retest this please |
|
not sure how to solve that test failure though |
|
retest this please |
|
I believe the problem is with the certs for |
|
OTOH I can't repro it on my laptop nor do browsers warn as far as I can tell. |
|
Ah, CA issue with the Jenkins worker. |
|
Pushed a commit to sickbay that domain for now. Following up separately. |
Rakefile
Outdated
| :check_html => true, | ||
| :file_ignore => [/javadoc/, /v2/, /pydoc/], | ||
| :url_ignore => [/jstorm.io/, /datatorrent.com/], | ||
| :url_ignore => [/jstorm.io/, /datatorrent.com/, /ai.google/], |
There was a problem hiding this comment.
Can you cite https://issues.apache.org/jira/browse/INFRA-16527 for removal of ai.google?
kennknowles
force-pushed
the
contribution-guide
branch
from
aeaa619 to
25a50fe
Compare
|
@asfgit merge |
|
Error: PR failed in verification; check the Jenkins job for more information. |
|
@asfgit merge |
6 participants
This does a few things:
http://apache-beam-website-pull-requests.storage.googleapis.com/438/contribute/index.html