-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
17 changed files
with
1,224 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,31 @@ | ||
--- | ||
name: Bug report | ||
about: Create a report to help us improve | ||
title: '' | ||
labels: bug, help wanted | ||
assignees: '' | ||
|
||
--- | ||
|
||
**Describe the bug** | ||
A clear and concise description of what the bug is. | ||
|
||
**To Reproduce** | ||
Steps to reproduce the behavior: | ||
1. Go to '...' | ||
2. Click on '....' | ||
3. Scroll down to '....' | ||
4. See error | ||
|
||
**Expected behavior** | ||
A clear and concise description of what you expected to happen. | ||
|
||
**Screenshots** | ||
If applicable, add screenshots to help explain your problem. | ||
|
||
**Desktop (please complete the following information):** | ||
- OS: [e.g. iOS] | ||
- Version [e.g. 22] | ||
|
||
**Additional context** | ||
Add any other context about the problem here. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
--- | ||
name: Feature request | ||
about: Suggest an idea for this project | ||
title: '' | ||
labels: enhancement, question | ||
assignees: '' | ||
|
||
--- | ||
|
||
**Is your feature request related to a problem? Please describe.** | ||
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...] | ||
|
||
**Describe the solution you'd like** | ||
A clear and concise description of what you want to happen. | ||
|
||
**Describe alternatives you've considered** | ||
A clear and concise description of any alternative solutions or features you've considered. | ||
|
||
**Additional context** | ||
Add any other context or screenshots about the feature request here. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
### Summary :memo: | ||
_Write an overview about it._ | ||
|
||
### Details | ||
_Describe more what you did on changes._ | ||
1. (...) | ||
2. (...) | ||
|
||
### Bugfixes :bug: (delete if dind't have any) | ||
- | ||
|
||
### Checks | ||
- [ ] Closed #798 | ||
- [ ] Tested Changes | ||
- [ ] Stakeholder Approval |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
version: 2 | ||
updates: | ||
- package-ecosystem: "github-actions" | ||
directory: "/" | ||
schedule: | ||
interval: "weekly" |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,24 @@ | ||
name: CodeStyle | ||
|
||
on: [push] | ||
|
||
jobs: | ||
build: | ||
runs-on: ubuntu-latest | ||
strategy: | ||
matrix: | ||
python-version: ["3.9", "3.10", "3.11", "3.12"] | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Set up Python ${{ matrix.python-version }} | ||
uses: actions/setup-python@v3 | ||
with: | ||
python-version: ${{ matrix.python-version }} | ||
- name: Install dependencies | ||
run: | | ||
set -x | ||
python -m pip install --upgrade pip | ||
pip install --user poetry | ||
make install | ||
- name: Check code style | ||
run: make lint |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,24 @@ | ||
name: UnitTest | ||
|
||
on: [push] | ||
|
||
jobs: | ||
build: | ||
runs-on: ubuntu-latest | ||
strategy: | ||
matrix: | ||
python-version: ["3.9", "3.10", "3.11", "3.12"] | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Set up Python ${{ matrix.python-version }} | ||
uses: actions/setup-python@v3 | ||
with: | ||
python-version: ${{ matrix.python-version }} | ||
- name: Install dependencies | ||
run: | | ||
set -x | ||
python -m pip install --upgrade pip | ||
pip install --user poetry | ||
make install | ||
- name: Check code style | ||
run: make test |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,83 @@ | ||
# Byte-compiled / optimized / DLL files | ||
__pycache__/ | ||
*.py[cod] | ||
*$py.class | ||
|
||
# Distribution / packaging | ||
.Python | ||
build/ | ||
develop-eggs/ | ||
dist/ | ||
downloads/ | ||
eggs/ | ||
.eggs/ | ||
lib/ | ||
lib64/ | ||
parts/ | ||
sdist/ | ||
var/ | ||
wheels/ | ||
dependency/**.jar | ||
pip-wheel-metadata/ | ||
share/python-wheels/ | ||
*.egg-info/ | ||
.installed.cfg | ||
*.egg | ||
MANIFEST | ||
|
||
# PyInstaller | ||
# Usually these files are written by a python script from a template | ||
# before PyInstaller builds the exe, so as to inject date/other infos into it. | ||
*.manifest | ||
*.spec | ||
|
||
# Installer logs | ||
pip-log.txt | ||
pip-delete-this-directory.txt | ||
|
||
# Unit test / coverage reports | ||
htmlcov/ | ||
.tox/ | ||
.nox/ | ||
.coverage | ||
.coverage.* | ||
.cache | ||
nosetests.xml | ||
coverage.xml | ||
*.cover | ||
*.py,cover | ||
.hypothesis/ | ||
.pytest_cache/ | ||
|
||
# pyenv | ||
.python-version | ||
|
||
# mkdocs documentation | ||
/site | ||
|
||
# pipenv | ||
# According to pypa/pipenv#598, it is recommended to include Pipfile.lock in version control. | ||
# However, in case of collaboration, if having platform-specific dependencies or dependencies | ||
# having no cross-platform support, pipenv may install dependencies that don't work, or not | ||
# install all needed dependencies. | ||
#Pipfile.lock | ||
|
||
# PEP 582; used by e.g. github.com/David-OConnor/pyflow | ||
__pypackages__/ | ||
|
||
# Environments | ||
.env | ||
.venv | ||
env/ | ||
venv/ | ||
ENV/ | ||
env.bak/ | ||
venv.bak/ | ||
|
||
# mypy | ||
.mypy_cache/ | ||
.dmypy.json | ||
dmypy.json | ||
|
||
# IDE | ||
.idea |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,143 @@ | ||
# How to develop on this project | ||
|
||
tosfs welcomes contributions from the community. | ||
|
||
**You need PYTHON3!** | ||
|
||
This instructions are for linux base systems. (Linux, MacOS, BSD, etc.) | ||
## Setting up your own fork of this repo. | ||
|
||
- On github interface click on `Fork` button. | ||
- Clone your fork of this repo. `git clone [email protected]:YOUR_GIT_USERNAME/tosfs.git` | ||
- Enter the directory `cd tosfs` | ||
- Add upstream repo `git remote add upstream https://github.com/volcengine/tosfs.git` | ||
|
||
## Setting up your own virtual environment | ||
|
||
Import the source code into your IDE(e.g. `Intellij IDEA`). And choose `poetry` to manage this project. | ||
If this step is successful, the IDE would create a virtual env for this project. | ||
Check it via `poetry env info -p`. | ||
|
||
## Install the project in develop mode | ||
|
||
Run `make install` to install the project in develop mode. | ||
|
||
## Run the tests to ensure everything is working | ||
|
||
Run `make test` to run the tests. | ||
|
||
## Create a new branch to work on your contribution | ||
|
||
Run `git checkout -b my_contribution` | ||
|
||
## Make your changes | ||
|
||
Edit the files using your preferred editor. (we recommend VIM or VSCode) | ||
|
||
## Format the code | ||
|
||
Run `make fmt` to format the code. | ||
|
||
## Run the linter | ||
|
||
Run `make lint` to run the linter. | ||
|
||
## Test your changes | ||
|
||
Run `make test` to run the tests. | ||
|
||
Ensure code coverage report shows `100%` coverage, add tests to your PR. | ||
|
||
## Build the docs locally | ||
|
||
Run `make docs` to build the docs. | ||
|
||
Ensure your new changes are documented. | ||
|
||
## Commit your changes | ||
|
||
The tosfs use multiple tools(e.g. `pylint`, `flake8`, `blank`...) to check and keep the code style for the project. | ||
So, before commit changes, we strongly recommand you run `make lint` and `make fmt` to verify or reformat your code. | ||
You can use git hook to run it automatically. Follow this: | ||
|
||
```shell | ||
cat > ${PROJECT_HOME}/.git/hooks/pre-commit << 'EOF' | ||
#!/bin/sh | ||
# run commands 'make lint' and 'make fmt' before committing | ||
echo "Running make lint and make fmt before commit..." | ||
make lint | ||
lint_exit_code=$? | ||
make fmt | ||
fmt_exit_code=$? | ||
# if 'make lint' or 'make fmt' failed, cancel commit | ||
if [ $lint_exit_code -ne 0 -o $fmt_exit_code -ne 0 ]; then | ||
echo "make lint or make fmt failed, aborting commit" | ||
exit 1 | ||
fi | ||
# if 'make lint' and 'make fmt' success, then commit | ||
exit 0 | ||
EOF | ||
|
||
chmod +x ${PROJECT_HOME}/.git/hooks/pre-commit | ||
``` | ||
|
||
This project uses [conventional git commit messages](https://www.conventionalcommits.org/en/v1.0.0/). | ||
|
||
Example: `fix(package): update core.py arguments` | ||
|
||
## Push your changes to your fork | ||
|
||
Run `git push origin my_contribution` | ||
|
||
## Submit a pull request | ||
|
||
On github interface, click on `Pull Request` button. | ||
|
||
Wait CI to run and one of the developers will review your PR. | ||
## Makefile utilities | ||
|
||
This project comes with a `Makefile` that contains a number of useful utility. | ||
|
||
```bash | ||
❯ make | ||
Usage: make <target> | ||
|
||
Targets: | ||
help: ## Show the help. | ||
show: ## Show the current environment. | ||
install: ## Install the project in dev mode. | ||
fmt: ## Format code using black & isort. | ||
lint: ## Run pep8, black, mypy linters. | ||
test: lint ## Run tests and generate coverage report. | ||
watch: ## Run tests on every change. | ||
clean: ## Clean unused files. | ||
release: ## Create a new tag for release. | ||
docs: ## Build the documentation. | ||
release_wheel: ## Release wheel for python client. | ||
``` | ||
|
||
## Making a new release | ||
|
||
This project uses [semantic versioning](https://semver.org/) and tags releases with `X.Y.Z` | ||
Every time a new tag is created and pushed to the remote repo, github actions will | ||
automatically create a new release on github and trigger a release on PyPI. | ||
|
||
For this to work you need to setup a secret called `PIPY_API_TOKEN` on the project settings>secrets, | ||
this token can be generated on [pypi.org](https://pypi.org/account/). | ||
|
||
To trigger a new release all you need to do is. | ||
|
||
1. If you have changes to add to the repo | ||
* Make your changes following the steps described above. | ||
* Commit your changes following the [conventional git commit messages](https://www.conventionalcommits.org/en/v1.0.0/). | ||
2. Run the tests to ensure everything is working. | ||
4. Run `make release` to create a new tag and push it to the remote repo. | ||
|
||
the `make release` will ask you the version number to create the tag, ex: type `0.1.1` when you are asked. | ||
|
||
> **CAUTION**: The make release will change local changelog files and commit all the unstaged changes you have. |
Oops, something went wrong.