Skip to content

Commit

Permalink
docs: Consolidate contributing info (#12109)
Browse files Browse the repository at this point in the history
  • Loading branch information
stinodego authored Oct 30, 2023
1 parent a8485b9 commit f3ee4d0
Show file tree
Hide file tree
Showing 11 changed files with 87 additions and 57 deletions.
1 change: 0 additions & 1 deletion Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
resolver = "2"
members = [
"crates/*",
"contribution/*",
"examples/*",
]
default-members = [
Expand Down
28 changes: 0 additions & 28 deletions contribution/README.md

This file was deleted.

8 changes: 0 additions & 8 deletions contribution/polars_ops_multiple_arguments/Cargo.toml

This file was deleted.

2 changes: 1 addition & 1 deletion docs/_build/snippets/under_construction.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
!!! warning ":construction: Under Construction :construction: "
!!! warning ":construction: Under Construction :construction:"

This section is still under development. Want to help out? Consider contributing and making a [pull request](https://github.com/pola-rs/polars) to our repository.
Please read our [Contribution Guidelines](https://github.com/pola-rs/polars/blob/main/CONTRIBUTING.md) on how to proceed.
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# Continuous integration setup
# Continuous integration

Polars uses GitHub Actions as its continuous integration (CI) tool. The setup is reasonably complex, as far as CI setups go. This document explains some of the design choices.
Polars uses GitHub Actions as its continuous integration (CI) tool. The setup is reasonably complex, as far as CI setups go. This page explains some of the design choices.

## Goal

Expand Down Expand Up @@ -40,6 +40,7 @@ However, since GitHub Actions does not allow sharing caches between feature bran

Care must also be taken not to exceed the maximum cache space of 10Gb allotted to open source GitHub repositories. Hence we do not do any caching on feature branches - we always use the cache available from the main branch. This also avoids any extra time that would be required to store the cache.

# Releases
## Releases

The release jobs for Rust and Python get triggered when a new release is published. Release drafter is used to automatically draft these releases. Refer to the [contributing guide](/CONTRIBUTING.md#release-flow) for the full release process.
The release jobs for Rust and Python are triggered manually.
Refer to the [contributing guide](./index.md#release-flow) for the full release process.
Original file line number Diff line number Diff line change
@@ -1,3 +1,29 @@
# Code style

This page contains some guidance on code style.

!!! info

Additional information will be added to this page later.

## Rust

### Naming conventions

Naming conventions for variables:

```rust
let s: Series = ...
let ca: ChunkedArray = ...
let arr: ArrayRef = ...
let arr: PrimitiveArray = ...
let dtype: DataType = ...
let data_type: ArrowDataType = ...
```

### Code example

```rust
use std::ops::Add;

use polars::export::arrow::array::*;
Expand Down Expand Up @@ -65,3 +91,4 @@ pub fn compute_expr_2_args(arg_1: &Series, arg_2: &Series) -> Series {
compute_chunked_array_2_args(ca_1, ca_2).into_series()
})
}
```
32 changes: 32 additions & 0 deletions docs/development/contributing/ide.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,32 @@
# IDE configuration

Using an integrated development environments (IDE) and configuring it properly will help you work on Polars more effectively.
This page contains some recommendations for configuring popular IDEs.

## Visual Studio Code

The following extensions are recommended.

### rust-analyzer

If you work on the Rust code at all, you will need the [rust-analyzer](https://marketplace.visualstudio.com/items?itemName=rust-lang.rust-analyzer) extension. This extension provides code completion for the Rust code.

For it to work well for the Polars code base, add the following settings to your `.vscode/settings.json`:

```json
{
"rust-analyzer.cargo.features": "all",
"rust-analyzer.linkedProjects": ["py-polars/Cargo.toml"],
}
```

### Ruff

The Ruff extension will help you conform to the formatting requirements of the Python code.
We use both the Ruff linter and formatter.

## PyCharm / RustRover / CLion

!!! info

More information needed.
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
render_macros: false
---

# Contributing
# Overview

Thanks for taking the time to contribute! We appreciate all contributions, from reporting bugs to implementing new features.
If you're unclear on how to proceed after reading this guide, please contact us on [Discord](https://discord.gg/4UfP5cfBE7).
Expand Down
26 changes: 14 additions & 12 deletions py-polars/tests/README.md → docs/development/contributing/test.md
Original file line number Diff line number Diff line change
@@ -1,17 +1,17 @@
# Polars test suite
# Test suite

This folder contains the main Polars test suite. This document contains some information on the various components of the test suite, as well as guidelines for writing new tests.
!!! info

The test suite contains four main components, each confined to their own folder: unit tests, parametric tests, benchmark tests, and doctests.
Additional information on the Rust test suite will be added to this page later.

Note that this test suite is indirectly responsible for testing Rust Polars as well. The Rust test suite is kept small to reduce compilation times. A lot of the Rust functionality is tested here instead.
The `py-polars/tests` folder contains the main Polars test suite.
This page contains some information on the various components of the test suite, as well as guidelines for writing new tests.

## Table of contents
The test suite contains four main components, each confined to their own folder: unit tests, parametric tests, benchmark tests, and doctests.

- [Unit tests](#unit-tests)
- [Parametric tests](#parametric-tests)
- [Doctests](#doctests)
- [Benchmark tests](#benchmark-tests)
Note that this test suite is indirectly responsible for testing Rust Polars as well.
The Rust test suite is kept small to reduce compilation times.
A lot of the Rust functionality is tested here instead.

## Unit tests

Expand All @@ -20,11 +20,13 @@ These tests are intended to make sure all Polars functionality works as intended

### Running unit tests

Run unit tests by running `make test` from the `py-polars` folder. This will compile the Rust bindings and then run the unit tests.
Run unit tests by running `make test` from the `py-polars` folder.
This will compile the Rust bindings and then run the unit tests.

If you're working in the Python code only, you can avoid recompiling every time by simply running `pytest` instead.

By default, slow tests are skipped. Slow tests are marked as such using a [custom pytest marker](https://docs.pytest.org/en/latest/example/markers.html).
By default, slow tests are skipped.
Slow tests are marked as such using a [custom pytest marker](https://docs.pytest.org/en/latest/example/markers.html).
If you wish to run slow tests, run `pytest -m slow`.
Or run `pytest -m ""` to run _all_ tests, regardless of marker.

Expand Down Expand Up @@ -103,7 +105,7 @@ The benchmark is somewhat cumbersome to run locally. You must first generate the

Make sure to install a release build of Polars before running the benchmark to guarantee the best results.

Refer to the [benchmark workflow](/.github/workflows/benchmark.yml) for detailed steps.
Refer to the [benchmark workflow](https://github.com/pola-rs/polars/blob/main/.github/workflows/benchmark.yml) for detailed steps.

### Running other benchmark tests

Expand Down
2 changes: 1 addition & 1 deletion docs/index.md
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ Polars has a very active community with frequent releases (approximately weekly)

## Contributing

We appreciate all contributions, from reporting bugs to implementing new features. Read our [contributing guide](development/contributing.md) to learn more.
We appreciate all contributions, from reporting bugs to implementing new features. Read our [contributing guide](development/contributing/index.md) to learn more.

## License

Expand Down
7 changes: 6 additions & 1 deletion mkdocs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,12 @@ nav:
- API reference: api/index.md

- Development:
- development/contributing.md
- Contributing:
- development/contributing/index.md
- development/contributing/ide.md
- development/contributing/test.md
- development/contributing/ci.md
- development/contributing/code-style.md
- development/versioning.md

- Releases:
Expand Down

0 comments on commit f3ee4d0

Please sign in to comment.