-
Notifications
You must be signed in to change notification settings - Fork 5
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #1 from alfcan/main
Refactoring, Added Italian Conversation, and Docker Porting of csDetector
- Loading branch information
Showing
58 changed files
with
3,693 additions
and
874 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,38 @@ | ||
name: Python Tests | ||
|
||
on: | ||
push: | ||
branches: [main, dev] | ||
pull_request: | ||
branches: [main, dev] | ||
|
||
jobs: | ||
test: | ||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- name: Checkout | ||
uses: actions/checkout@v2 | ||
|
||
- name: Set up Python 3.8 | ||
uses: actions/setup-python@v2 | ||
with: | ||
python-version: 3.8 | ||
|
||
- name: Install dependencies | ||
run: | | ||
python -m pip install --upgrade pip | ||
pip install -r requirements.txt | ||
- name: Test with pytest and Coverage | ||
run: | | ||
pip install coverage | ||
pytest tests --cov=src --cov-branch | ||
env: | ||
CI: true | ||
|
||
- name: Check Branch Coverage | ||
run: | | ||
coverage report --fail-under=75 | ||
env: | ||
CI: true |
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,10 @@ | ||
{ | ||
"python.testing.pytestArgs": ["tests"], | ||
"python.testing.unittestEnabled": false, | ||
"python.testing.pytestEnabled": true, | ||
"[python]": { | ||
"editor.defaultFormatter": "ms-python.autopep8" | ||
}, | ||
"python.formatting.provider": "none", | ||
"python.analysis.typeCheckingMode": "basic" | ||
} |
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,89 @@ | ||
# CADOCS | CONTRIBUTING | ||
|
||
## Commits | ||
|
||
For contributing commit messagges must follow the standard of [Conventional Commits](https://www.conventionalcommits.org/). | ||
|
||
Part of the following document is taken from https://github.com/angular/angular/blob/22b96b9/CONTRIBUTING.md#-commit-message-guidelines | ||
|
||
### Commit Message Format | ||
|
||
Each commit message consists of a `header`, a `body` and a `footer`. The header has a special format that includes a `type`, a `scope` and a `subject`: | ||
|
||
``` | ||
<type>(<scope>): <summary> | ||
<BLANK LINE> | ||
<body> | ||
<BLANK LINE> | ||
<footer> | ||
``` | ||
|
||
The **header** is mandatory and the **scope** of the header must be omitted if the change is common among the packages, otherwise it's mandatory. | ||
|
||
``` | ||
<type>(<scope>): <description> | ||
│ │ │ | ||
│ │ └─⫸ A brief description of the commit | ||
│ │ | ||
│ └─⫸ Commit Scope: api | service | chatbot | intent | ||
│ | ||
└─⫸ Commit Type: build|ci|docs|feat|fix|refactor|impr|style|test | ||
``` | ||
|
||
#### Description | ||
|
||
- use the imperative, present tense: "change" not "changed" nor "changes" | ||
- don't capitalize the first letter | ||
- no dot (.) at the end | ||
|
||
#### Type | ||
|
||
Must be one of the following: | ||
|
||
- build: Changes that affect the build system or external dependencies | ||
- ci: Changes to our CI configuration files and scripts | ||
- docs: Documentation only changes | ||
- feat: A new feature | ||
- fix: A bug fix | ||
- refactor: A code change that neither fixes a bug nor adds a feature | ||
- impr: Improve an existing feature or existing dataset | ||
- style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc) | ||
- test: Adding missing tests or correcting existing tests | ||
|
||
### Scope | ||
|
||
The scope must be the project section within the change is happening. | ||
|
||
Supported scopes: | ||
|
||
- `api`: scope to handling api connection for platforms | ||
- `service`: scope to define support methods for complex functionalities | ||
- `chatbot`: scope to handling chatbot functionalities | ||
- `intent`: scope to resolve user intents | ||
|
||
### Body | ||
|
||
The **body** is optional. Just as in the **description**, use the imperative, present tense: "fix" not "fixed" nor "fixes". | ||
The body should extend the content of the description. | ||
|
||
### Footer | ||
|
||
The footer is optional. The footer should contain the GitHub issue reference that this commit **Closes**. | ||
|
||
## Branches | ||
|
||
New branches must follow this format for their names: `<type>/<scope>/<description>` | ||
|
||
- `type` and `scope` refer to the ones described in the Commits section. | ||
- `description` must be a word representing the changes happening in the branch. | ||
|
||
### Branching strategy | ||
|
||
The branches will be defined as follows: a _main long branch_, named `dev`, which serves as the development branch; starting from this branch there will be four other long branches, each dedicated to a specific change request (CR). These branches will be named after the identifiers of the corresponding CRs. | ||
|
||
In addition to the long branches, we will also use _short branches_, which are temporary branches created from the specific long branch of each CR and have a limited scope and duration. | ||
Whenever a developer starts working on a specific feature or activity related to a CR, he or she creates a short branch for that particular activity. | ||
|
||
Once the activity is completed, the changes will be merged into the corresponding _long branch_ (i.e., the CR-specific branch). | ||
|
||
Once the integration of all subfunctions is complete, the CR-specific long branch will be merged with the `dev` branch. |
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
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
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,2 @@ | ||
[pytest] | ||
pythonpath = src |
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,9 @@ | ||
Flask==2.3.2 | ||
langdetect==1.0.9 | ||
pytest==7.4.0 | ||
pytest-cov==4.1.0 | ||
pytest-mock==3.11.1 | ||
python-dotenv==1.0.0 | ||
requests==2.31.0 | ||
slackclient==2.9.4 | ||
slackeventsapi==3.0.1 |
Binary file not shown.
Empty file.
Empty file.
Oops, something went wrong.