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

Merge CONTRIBUTING.md into website contributing guide. #15449

Merged
merged 6 commits into from
Nov 9, 2023

Conversation

ScottTodd
Copy link
Member

@ScottTodd ScottTodd commented Nov 6, 2023

Following up on some ideas noted at #15116 (comment).

We had previously split "contributing" docs between the root CONTRIBUTING.md and a separate developer doc. Now that the developer doc is published on our website at https://iree.dev/developers/general/contributing/, we can move more of the documentation there and just have the root file point at the website.

Preview for review: https://scotttodd.github.io/iree/developers/general/contributing/

  • I rewrote and reordered most of the sections
  • I have more ideas for how to streamline these docs... trying to not spent too much time on this right now though 😛
    • Separate "infrastructure" page explaining how to interact with the CI (build/test/benchmark/etc.)
    • Deemphasize "contributor tips" and/or add more tips (like how to set up Visual Studio Code)

I took some inspiration from https://llvm.org/docs/Contributing.html and https://llvm.org/docs/DeveloperPolicy.html.

@ScottTodd ScottTodd added the documentation ✏️ Improvements or additions to documentation label Nov 6, 2023
@ScottTodd ScottTodd requested a review from jpienaar November 6, 2023 23:51
Comment on lines 104 to 106
### :octicons-git-merge-16: Obtaining commit access

!!! note "TODO(jpienaar): write this :)"
Copy link
Member Author

Choose a reason for hiding this comment

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

(I can also write this once you have the form ready to link to)

Should maybe also add a link to that in https://github.com/openxla/community ?

Copy link
Collaborator

@stellaraccident stellaraccident left a comment

Choose a reason for hiding this comment

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

One statement I would like added.

someone else to merge it for them (e.g. because they don't have write access
yet).

### :octicons-git-merge-16: Obtaining commit access
Copy link
Member Author

Choose a reason for hiding this comment

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

Wrote this section :)

Live preview: https://scotttodd.github.io/iree/developers/general/contributing/#obtaining-commit-access

image

The form goes to Jacques to review access requests

@ScottTodd ScottTodd merged commit e2f8dc7 into iree-org:main Nov 9, 2023
45 checks passed
@ScottTodd ScottTodd deleted the contributing-guide branch November 9, 2023 20:02
ramiro050 pushed a commit to ramiro050/iree that referenced this pull request Dec 19, 2023
Following up on some ideas noted at
iree-org#15116 (comment).

We had previously split "contributing" docs between the root
`CONTRIBUTING.md` and a separate developer doc. Now that the developer
doc is published on our website at
https://iree.dev/developers/general/contributing/, we can move more of
the documentation there and just have the root file point at the
website.

Preview for review:
https://scotttodd.github.io/iree/developers/general/contributing/

* I rewrote and reordered most of the sections
* I have more ideas for how to streamline these docs... trying to not
spent too much time on this right now though 😛
* Separate "infrastructure" page explaining how to interact with the CI
(build/test/benchmark/etc.)
* Deemphasize "contributor tips" and/or add more tips (like how to set
up Visual Studio Code)

I took some inspiration from https://llvm.org/docs/Contributing.html and
https://llvm.org/docs/DeveloperPolicy.html.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation ✏️ Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants