This is an automated email from the ASF dual-hosted git repository.
timbrown pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-xtable.git
The following commit(s) were added to refs/heads/main by this push:
new c64e94d6 Add contribution guidelines to reflect stale PR's guidelines
c64e94d6 is described below
commit c64e94d62a1b3fe0981c21690849a39c603828b0
Author: Vinish Reddy <[email protected]>
AuthorDate: Mon Apr 7 02:46:46 2025 -0700
Add contribution guidelines to reflect stale PR's guidelines
---
README.md | 2 +-
website/community/sync.md | 22 ++++++++++++++++++++++
2 files changed, 23 insertions(+), 1 deletion(-)
diff --git a/README.md b/README.md
index 12b7ae25..d7ff6447 100644
--- a/README.md
+++ b/README.md
@@ -140,7 +140,7 @@ docker run \
For setting up the repo on IntelliJ, open the project and change the Java
version to Java 11 in File->ProjectStructure
-You have found a bug, or have a cool idea you that want to contribute to the
project ? Please file a GitHub issue
[here](https://github.com/apache/incubator-xtable/issues)
+Found a bug or have a cool idea to contribute? Open a GitHub
[issue](https://github.com/apache/incubator-xtable/issues) to get started. For
more contribution guidelines and ways to stay involved, visit our
[community](https://xtable.apache.org/community/sync) page.
## Adding a new target format
Adding a new target format requires a developer implement
[ConversionTarget](./xtable-api/src/main/java/org/apache/xtable/spi/sync/ConversionTarget.java).
Once you have implemented that interface, you can integrate it into the
[ConversionController](./xtable-core/src/main/java/org/apache/xtable/conversion/ConversionController.java).
If you think others may find that target useful, please raise a Pull Request
to add it to the project.
diff --git a/website/community/sync.md b/website/community/sync.md
index eda95572..54d0d68e 100644
--- a/website/community/sync.md
+++ b/website/community/sync.md
@@ -14,6 +14,28 @@ Here's a quick view of the upcoming community syncs:
## Join the Community on Slack
We have a [Slack
channel](https://join.slack.com/t/apachextablei-yoi8504/shared_invite/zt-2y9hqskhv-ZfSwoxzq~PTJlWGj0V1sQw)
that you can join for help with any questions, issues, design choices,
announcements, etc.
+## Contribution Guidelines
+
+### Getting Started
+
+- Fork the repo and create a feature branch from `main`.
+- Open a pull request (PR) against the main branch with a clear summary of
your changes.
+ - Use descriptive commit messages.
+ - Write or update unit tests where applicable.
+- Refer to the
[README](https://github.com/apache/incubator-xtable/blob/main/README.md) for
more detailed setup instructions.
+
+### Pull Request Review Process
+
+- All PRs are reviewed by project committers.
+- Review comments may request changes, please try to update the PR within a
reasonable time.
+- We aim to keep the review loop tight and collaborative.
+
+### Handling Stale Pull Requests
+
+- To keep development moving and reduce backlog, PRs with no activity for many
days may be marked as stale.
+- A committer may pick up a stale PR, incorporate necessary changes (including
rebasing or completing the implementation), and merge it.
+- Attribution to the original author will always be preserved in the commit
history. This helps prevent good contributions from getting lost due to
inactivity, while staying aligned with Apache’s community-first philosophy.
+
## Recordings of the Community Calls
Find date-wise recordings here.
