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

[BEAM-13991] [Website] update contribute to beam page #17136

Merged

Conversation

bullet03
Copy link
Contributor

@bullet03 bullet03 commented Mar 21, 2022

[BEAM-13991]

NB!!! Firstly, PR #17089 ([BEAM-13992]) is needed to be merged, because it contains changes that will be used in current PR.

  • create a new landing page
  • transfer old landing page to 'get starting contributing' from left nav menu

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

  • Choose reviewer(s) and mention them in a comment (R: @username).
  • Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
  • Update CHANGES.md with noteworthy changes.
  • If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md

GitHub Actions Tests Status (on master branch)

Build python source distribution and wheels
Python tests
Java tests

See CI.md for more information about GitHub Actions CI.

@bullet03
Copy link
Contributor Author

R: @AydarZaynutdinov


.list-text-size-md
li
font-size: 14px
Copy link
Contributor

Choose a reason for hiding this comment

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

Add an empty line.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done


Apache Beam is an open source project developed and maintained by a friendly community of users, contributors, committers, and project management committee (PMC) members.

Everyone is welcome to join and contribute to the project in multiple ways, not only with code contributions. Contributions like asking questions, reporting bugs, proposing new features, improving documentation, adding to the website, organizing events, or writing blog posts among other contributions, are welcome and recognized.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Everyone is welcome to join and contribute to the project in multiple ways, not only with code contributions. Contributions like asking questions, reporting bugs, proposing new features, improving documentation, adding to the website, organizing events, or writing blog posts among other contributions, are welcome and recognized.
Everyone is welcome to join and contribute to the project in multiple ways, not only with code contributions. Contributions like asking questions, reporting bugs, proposing new features, improving documentation, adding to the website, organizing events, or writing blog posts among other contributions, are welcome and recognized.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done


Everyone is welcome to join and contribute to the project in multiple ways, not only with code contributions. Contributions like asking questions, reporting bugs, proposing new features, improving documentation, adding to the website, organizing events, or writing blog posts among other contributions, are welcome and recognized.

### What’s In It for You?
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
### What’s In It for You?
### What’s In It for You?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

### What’s In It for You?
- Become a part of the world’s largest, most welcoming open source community
- Drive innovation for next-gen distributed data processing
- Make an impact by bringing advanced technology solutions into reality
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Make an impact by bringing advanced technology solutions into reality
- Make an impact by bringing advanced technology solutions into reality

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

<tr>
<td>Improve documentation</td>
<td>
Click the edit icon in our documentation and it’ll take you to our <a href="https://github.com/apache/beam/">GitHub</a>, where you can automatically make the change you want and send a Pull Request.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Click the edit icon in our documentation and it’ll take you to our <a href="https://github.com/apache/beam/">GitHub</a>, where you can automatically make the change you want and send a Pull Request.
Click the edit icon in our documentation and it’ll take you to our <a href="https://github.com/apache/beam/">GitHub</a>, where you can automatically make the change you want and send a Pull Request.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

<ol>
<li>Create a Jira ticket and mark it as a feature request</li>
<br>OR<br><br>
<li>Send an email to <a href="mailto:dev@beam.apache.org">dev@beam.apache.org</a> with subject: [Feature Proposal] <your idea>.</li>
Copy link
Contributor

Choose a reason for hiding this comment

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

with subject: [Feature Proposal] <your idea>. isn't shown on the page in right way:

image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Comment on lines 180 to 181
“Knowledge sharing and collaboration is something that comes naturally in the community. Together, we can work with the most complex problems and just solve them.
”Matt Casters, Co-founder @ Apache Hop
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
“Knowledge sharing and collaboration is something that comes naturally in the community. Together, we can work with the most complex problems and just solve them.
Matt Casters, Co-founder @ Apache Hop
“Knowledge sharing and collaboration is something that comes naturally in the community. Together, we can work with the most complex problems and just solve them.
Matt Casters, Co-founder @ Apache Hop

Matt Casters, Co-founder @ Apache Hop should be on the right side
Also need to add photo

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

”Matt Casters, Co-founder @ Apache Hop

“I really enjoy working with Beam. The understanding of its internals empowers you to fine-tune the open source, customize it. Beam is very powerful and the abstraction is its most significant feature.”
Talat Ulyarer, Sr Principal Software Engineer @ Palo Alto Networks
Copy link
Contributor

Choose a reason for hiding this comment

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

Talat Ulyarer, Sr Principal Software Engineer @ Palo Alto Networks should be on the right side
Also need to add photo

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Talat Ulyarer, Sr Principal Software Engineer @ Palo Alto Networks

“My takeaway: everyone should join the [Apache Beam] mailing lists! This is where all the discussions happen, and it is a great place for learning. I also encourage everyone to attend and participate in conferences to meet and connect with the community.”
Tobias Kaymak, Senior Data Engineer @ Ricardo
Copy link
Contributor

Choose a reason for hiding this comment

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

Tobias Kaymak, Senior Data Engineer @ Ricardo should be on the right side
Also need to add photo

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@@ -0,0 +1,389 @@
---
Copy link
Contributor

Choose a reason for hiding this comment

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

As I can see you add Beam Contribution Guide page (rewrite from previous one). Make sure that [BEAM-13992] changes are also included

Copy link
Contributor Author

Choose a reason for hiding this comment

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

added a comment to PR description to merge 13992 first and only after that 13991

border-radius: 50%

.no-bullet
list-style-type: none
Copy link
Contributor

Choose a reason for hiding this comment

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

Add an empty line.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done


- ask or answer questions on [user@beam.apache.org](/community/contact-us/) or
[stackoverflow](https://stackoverflow.com/questions/tagged/apache-beam)
- review proposed design ideas on [dev@beam.apache.org](/community/contact-us/)
Copy link
Contributor

Choose a reason for hiding this comment

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

Change links to full format: https....

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

<td>Improve the website</td>
<td class="list-text-size-md">
<ul>
<li>Follow the website <a href="https://beam.apache.org/contribute/">Contribution Guide</a> to make changes yourself.</li>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<li>Follow the website <a href="https://beam.apache.org/contribute/">Contribution Guide</a> to make changes yourself.</li>
<li>Follow the website <a href="https://beam.apache.org/contribute/get-started-contributing/">Contribution Guide</a> to make changes yourself.</li>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

<ul>
<li>Create and contribute a custom <a href="https://beam.apache.org/documentation/programming-guide/#pipeline-io">I/O connector</a> for a specific data storage system you want to connect to, if it is not on <a href="https://beam.apache.org/documentation/io/built-in/">the I/O list yet</a>.</li>
<li>Look at our project backlog to get some inspiration on what to build.</li>
<li>Follow our <a href="https://beam.apache.org/contribute/">Code Contribution Guide</a>.</li>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<li>Follow our <a href="https://beam.apache.org/contribute/">Code Contribution Guide</a>.</li>
<li>Follow our <a href="https://beam.apache.org/contribute/get-started-contributing/">Code Contribution Guide</a>.</li>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

<td>Share a code sample or template</td>
<td class="list-text-size-md">
<ol>
<li>Read our <a href="https://beam.apache.org/contribute/">Code Contribution Guide</a>.</li>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<li>Read our <a href="https://beam.apache.org/contribute/">Code Contribution Guide</a>.</li>
<li>Read our <a href="https://beam.apache.org/contribute/get-started-contributing/">Code Contribution Guide</a>.</li>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

<ol>
<li>Read our <a href="https://beam.apache.org/contribute/">Code Contribution Guide</a>.</li>
<li>Create your code sample or template. </li>
<li><a href="https://beam.apache.org/contribute/#make-your-change">Make a Pull Request</a> to add it to the Beam example folder.</li>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<li><a href="https://beam.apache.org/contribute/#make-your-change">Make a Pull Request</a> to add it to the Beam example folder.</li>
<li><a href="https://beam.apache.org/contribute/get-started-contributing/#make-your-change">Make a Pull Request</a> to add it to the Beam example folder.</li>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Comment on lines 180 to 184
“Knowledge sharing and collaboration is something that comes naturally in the community. Together, we can work with the most complex problems and just solve them.”
<div class="contribute-quote-block">
<span>Matt Casters, Co-founder @ Apache Hop</span>
<img src="https://beam.apache.org/images/matt_casters_photo.png" alt="Matt Casters">
</div>
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you please change this part like it is done on the Case Studies Page (visual)

image

the same for all other quotes.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done


## The Traits of an Apache Beam Committer

{{< contributor/row_of_traits committer_traits >}}
Copy link
Contributor

Choose a reason for hiding this comment

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

This section looks like:

image

Could we change style of the Learn more., so it looks like in Case Studies Page

image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@dariabezkorovaina
Copy link
Contributor

@pcoet please take a look:)

# Conflicts:
#	website/www/site/assets/scss/main.scss
@tkaymak
Copy link
Contributor

tkaymak commented Mar 25, 2022

👍

<td class="list-text-size-md">
<ul>
<li>Follow the website <a href="https://beam.apache.org/contribute/get-started-contributing/">Contribution Guide</a> to make changes yourself.</li>
<li>Check our <a href="https://issues.apache.org/jira/browse/BEAM-13229?jql=project%20%3D%20BEAM%20AND%20resolution%20%3D%20Unresolved%20AND%20labels%20%3D%20website%20ORDER%20BY%20priority%20DESC%2C%20updated%20DESC">Website Backlog</a> in Jira to find rooms for improvement areas, or propose one.</li>
Copy link
Collaborator

Choose a reason for hiding this comment

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

"find rooms for improvement areas" -> this doesn't quite make sense to me (?)

Copy link
Contributor

Choose a reason for hiding this comment

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

@bullet03 please change to "find rooms for improvement, or propose one."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

changed


Even though there is no strict protocol to become a committer, the most important thing we look for is that you display the traits of an Apache Beam committer and that you have been consistently contributing to the project and community.

Candidates for new committers are suggested by current committers or PMC members, and voted upon by the PMC team. So if you would like to become a committer, engage with the community and keep contributing to Apache Beam, you might also want to talk to other committers and ask for their advice and guidance. Feel free to email any committer directly and ask for their advice!
Copy link
Collaborator

Choose a reason for hiding this comment

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

"Apache Beam, you might" -> "Apache Beam. You might"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

changed


Since the PMC members are responsible for establishing the Apache Beam vision and leading our community to work in harmony to achieve it, we look for committed individuals who can perform the responsibilities of the PMC, as well as lead our community to fulfill its vision. This is why we strive to build a diverse PMC that complements skills and expertise sharing the same sets of values.

Ideally, we find candidates among active community members that have shown initiative to shape the direction of Beam and have shown willingness to learn the official processes, such as how to create or verify for releases, how to ensure trademark compliance for events, among others. We also expect PMC candidates to be level-headed, constructive, supportive, and willing to “disagree and commit” at times.
Copy link
Collaborator

Choose a reason for hiding this comment

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

"verify for releases" -> "verify releases"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

changed

1. Clone the git repository. You can download it anywhere you like.

$ mkdir -p ~/go/src/github.com/apache
$ cd ~/go/src/github.com/apache
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you confirm that these commands are right? I don't understand why you'd create a directory at ~/go/src/github.com/apache before cloning the repo. Unless this is something specific for Go...?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i was confirmed, that they are correct.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think this should only be necessary if you're doing Go development. Also, it doesn't mention forking the repo first. I think the steps are fixed in the other PR, but please make sure...

$ go get -u ./...

1. Check the environment was set up correctly.
- **Option 1**: validate the Go, Java, and Python environments:
Copy link
Collaborator

Choose a reason for hiding this comment

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

The Markdown is not rendering correctly in any of these steps.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is changed in PR#17098 (Beam 13992) #17089.
13992 should be merged before this PR, because it contains changes, that will be used in current PR, as it is stated in description

export GOLANG_PROTOBUF_REGISTRATION_CONFLICT=ignore
./gradlew :sdks:go:examples:wordCount
```
- For **Python development**:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Markdown not rendering...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is changed in PR#17098 (Beam 13992) #17089.
13992 should be merged before this PR, because it contains changes, that will be used in current PR, as it is stated in description

```
./gradlew :sdks:python:wordCount
```
- For **Java development**:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please review the markdown in all of these steps. It's being treated as code, so e.g. **foo** is not rendering.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is changed in PR#17098 (Beam 13992) #17089.
13992 should be merged before this PR, because it contains changes, that will be used in current PR, as it is stated in description

@dariabezkorovaina
Copy link
Contributor

@pcoet I think we implemented all your comments, the markdown is fixed in PR # 17098, thank you for the review! Can you please take another look?

@bullet03 bullet03 changed the title Beam 13991 update contribute to beam page [Beam 13991] [Website] update contribute to beam page Apr 4, 2022
@bullet03 bullet03 changed the title [Beam 13991] [Website] update contribute to beam page [BEAM-13991] [Website] update contribute to beam page Apr 4, 2022
# Conflicts:
#	website/www/site/content/en/contribute/_index.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants