Submitting Documentation
To submit additions or corrections to Vaadin documentation, you need to create a pull request in the Vaadin Docs GitHub repository.
See Contributing to Projects for general instructions on contributing to Vaadin. You should separate enhancements, fixes, and new features into different pull requests. Check your changes for basic style violations.
Making a Pull Request
To submit a contribution, you need to create a feature branch and push that for review.
Contributors
As a contributor, you need to:
Fork the
https://github.com/vaadin/docs
repository in GitHub.Clone it locally to edit it.
Show code
bash
Expand code
$ git clone https://github.com/vaadin/docs.git
Make a new branch for the improvements.
Show code
bash
Expand code
$ git checkout -b my-feature-branch
Make the changes.
Show code
bash
Expand code
$ git add [files]
$ git commit -m "Fixed several typos in Flow documentation"
Push the changes to the branch in your forked repository.
Show code
bash
Expand code
$ git push --set-upstream origin submitting-to-docs
Create a pull request in the Docs repository.
Open the your fork repository page at GitHub in browser.
Your change should show there.
Click Compare & pull request.
In the Open a pull request form, enter a title for the PR, an optional comment, and click Create pull request.
Make and push fixes requested in the review.
Note that you do not need to set the upstream again, but can just
git push
the changes.Show code
bash
Expand code
$ git add ...
$ git commit -m "Fixes from review"
$ git push
Vaadin Developers
Vaadin developers with write access to the Docs repository should (like above):
Clone the
https://github.com/vaadin/docs
repository to edit it.Make a new branch for the improvements.
Make the changes.
- Preview locally while writing.
Push the changes to the new branch in the Docs repository.
Create a pull request in the Docs repository.
Open the Docs repository page in browser.
Your change should show there.
Click Compare & pull request.
In the Open a pull request form, enter a title for the PR, an optional comment, and click Create pull request.
Add reviewers
Review in the development team.
Component code examples (TypeScript and Java) need to be approved by
tomivirkki
,DiegoCardoso
, oryuriy-fix
.Can add
magi42
for documentation style review, or ask to do separately outside the development time box.No dedicated proofreader is currently available, but
magi42
can be asked for review.
Make fixes requested in the review.
Merge the PR by yourself or have the team lead or main author do it.
Requirements for Merging
Merging a pull request has the following requirements:
The PR must have been accepted in review
Deployment preview build must finish successfully
You must have signed the Contributor License Agreement
The following requirements are optional and not enforced:
- Vale checks are successful
Merging
For contributions, the reviewer merges the pull request once it has been approved.
Vaadin developers with write access to the repository are able to merge their own pull request once it is reviewed and all checks are successful. A reviewer should not merge the PR, unless it is obvious that the author does not have more upcoming additions. This gives the possibility to get comments for new documentation, while keeping on writing it.