Skip to content

Add issue template and docs for iceberg proposals#9932

Merged
danielcweeks merged 7 commits into
apache:mainfrom
danielcweeks:improvement-proposals
Mar 20, 2024
Merged

Add issue template and docs for iceberg proposals#9932
danielcweeks merged 7 commits into
apache:mainfrom
danielcweeks:improvement-proposals

Conversation

@danielcweeks
Copy link
Copy Markdown
Contributor

This PR adds an issue template for Iceberg improvement proposals and adds docs to the site to cover the proposal process.

@danielcweeks
Copy link
Copy Markdown
Contributor Author

@jbonofre I'm not sure why I can't add you as a reviewer.

@ajantha-bhat
Copy link
Copy Markdown
Member

ajantha-bhat commented Mar 12, 2024

@jbonofre I'm not sure why I can't add you as a reviewer.

What I observed is only collaborators, committers can be added as reviewer.
https://github.com/apache/iceberg/blob/main/.asf.yaml

So, I manually CC them in the PR for review if required.

Copy link
Copy Markdown
Contributor

@liurenjie1024 liurenjie1024 left a comment

Choose a reason for hiding this comment

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

Thanks for this, just added some minor nits.

Comment thread .github/ISSUE_TEMPLATE/iceberg_proposal.yml
Comment thread site/docs/contribute.md Outdated
Comment thread site/docs/contribute.md Outdated
@jbonofre
Copy link
Copy Markdown
Member

@danielcweeks That's because my user is not in the collaborators list in .asf.yml. Let me check with asfinfra if we can have more than 10 entries in collaborators.

Comment thread .github/ISSUE_TEMPLATE/iceberg_proposal.yml Outdated
Comment thread site/docs/contribute.md
Comment thread .github/ISSUE_TEMPLATE/iceberg_proposal.yml Outdated
danielcweeks and others added 5 commits March 13, 2024 18:36
Co-authored-by: Renjie Liu <liurenjie2008@gmail.com>
Co-authored-by: Renjie Liu <liurenjie2008@gmail.com>
Co-authored-by: Eduard Tudenhoefner <etudenhoefner@gmail.com>
Comment thread .github/ISSUE_TEMPLATE/iceberg_proposal.yml
Comment thread site/docs/contribute.md Outdated
Comment thread site/docs/contribute.md
Copy link
Copy Markdown
Contributor

@Fokko Fokko left a comment

Choose a reason for hiding this comment

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

Looks great!

Copy link
Copy Markdown
Member

@RussellSpitzer RussellSpitzer left a comment

Choose a reason for hiding this comment

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

LGTM

@danielcweeks danielcweeks merged commit 7151401 into apache:main Mar 20, 2024
zachdisc pushed a commit to zachdisc/iceberg that referenced this pull request Dec 23, 2024
* Add issue template and docs for iceberg proposals

* Add vote clarification

* Update site/docs/contribute.md

Co-authored-by: Renjie Liu <liurenjie2008@gmail.com>

* Update site/docs/contribute.md

Co-authored-by: Renjie Liu <liurenjie2008@gmail.com>

* Update .github/ISSUE_TEMPLATE/iceberg_proposal.yml

Co-authored-by: Eduard Tudenhoefner <etudenhoefner@gmail.com>

* Fix issue template link

* Address comments

---------

Co-authored-by: Renjie Liu <liurenjie2008@gmail.com>
Co-authored-by: Eduard Tudenhoefner <etudenhoefner@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants