We welcome contribution from everyone in the community. All communications in this repository will be in English.
Every contributor to es-toolkit should adhere to our Code of Conduct. Please read the full text to understand what actions will and will not be tolerated.
Note that we value performance, simplicity of implementation, and detailed documentations. We do not aim for supporting a variety of features and options. Our goal is to provide a small set of performant and well-functioning utilities.
All functions es-toolkit provides should be more performant than or similar with that of alternative libraries provide.
We measure the performance of our library every time our code is edited. We are using vitest's benchmark feature. For our benchmark code, please refer to our benchmark directory.
When a new functionality is added, a benchmark code should be added. Please add screenshots of the benchmarks when opening a pull request for easy reference and history tracking.
We value implementation and interface simplicity over a variety of features for performance, code readability, and easy maintenance. Our functions will not provide complex options to suit every usecase.
In this manner, instead of having complex options of making full use of overloading, etc, to support edge cases, we aim to provide the most simplest interface and implementation for the most common 85% usecases.
We recognize that there are multiple approaches to achieving the same functionality. If the performance difference is less than 10%, please follow our coding style guidelines:
1. Prefer for loops over reduce.
In most cases, we prefer using for loops over reduce. This is because maintaining immutability with reduce can be challenging without tools like immer, and functional programming typically allows local mutability.
For instance, we prefer implementing keyBy using a for ... of loop instead of reduce.
export function keyBy<T, K extends PropertyKey>(arr: readonly T[], getKeyFromItem: (item: T) => K): Record<K, T> {
const result = {} as Record<K, T>;
for (const item of arr) {
const key = getKeyFromItem(item);
result[key] = item;
}
return result;
}All of our functions should be documented in detail for easy reference. All functions should have the jsdoc and corresponding documents in our documentation directory for all of their features.
We use English as our primary language, but we aim to support Korean documents in our best effort. If you have difficulties writing Korean documents, please let our contributors know so that we can provide the corresponding Korean documents for you.
You can contribute to es-toolkit via:
- Improving our docs
- Reporting a bug in our issues tab
- Requesting a new feature or package
- Having a look at our issue list to see what's to be fixed
You can raise your own pull request. The title of your pull request should match the following format:
<type>[function names]: <description>
We do not care about the number, or style of commits in your history, because we squash merge every PR into main.
Feel free to commit in whatever style you feel comfortable with.
Type must be one of those
if you changed shipped code :
- feat - for any new functionality additions
- fix - for any fixes that don't add new functionality
if you haven't changed shipped code :
- docs - if you only change documentation
- test - if you only change tests
other :
- chore - anything else
The name of function that you made changes. (ex: debounce, throttle)
If you made changes across multiple packages, writing package scope is optional.
A clear and concise description of what the pr is about.