-
Notifications
You must be signed in to change notification settings - Fork 639
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add CI to validate URLs and enhance Python code quality (#263)
* add CI and validate urls Signed-off-by: jacky82226 <[email protected]> * fix flake8 failure Signed-off-by: jacky82226 <[email protected]> * prefer https:// instead of http:// Signed-off-by: jacky82226 <[email protected]> * fix broken urls Signed-off-by: jacky82226 <[email protected]> * remove retired MXNetONNXExport.ipynb Signed-off-by: jacky82226 <[email protected]> * move workflow yaml file and add pull_request_template.md Signed-off-by: jacky82226 <[email protected]> * decouple to validate_all_urls and validate_changed_urls Signed-off-by: jacky82226 <[email protected]> * checkout repo Signed-off-by: jacky82226 <[email protected]> * add -- in the end to prevent ambiguous main Signed-off-by: jacky82226 <[email protected]> * git fetch --all Signed-off-by: jacky82226 <[email protected]> * origin/main Signed-off-by: jacky82226 <[email protected]> * ignore .pyc Signed-off-by: Chun-Wei Chen <[email protected]> * add apache-2.0 Signed-off-by: Chun-Wei Chen <[email protected]>
- Loading branch information
Showing
26 changed files
with
1,863 additions
and
1,940 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,6 @@ | ||
**Description** | ||
- Describe your changes. | ||
|
||
**Motivation and Context** | ||
- Why is this change required? What problem does it solve? | ||
- If it fixes an open issue, please link to the issue 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,43 @@ | ||
name: validate | ||
|
||
on: | ||
schedule: | ||
- cron: '00 00 * * MON' | ||
push: | ||
branches: [main] | ||
pull_request: | ||
branches: [main] | ||
workflow_dispatch: | ||
|
||
jobs: | ||
build: | ||
runs-on: ubuntu-latest | ||
strategy: | ||
matrix: | ||
python-version: ['3.9',] | ||
architecture: ['x64'] | ||
steps: | ||
- name: Checkout onnx/tutorials | ||
uses: actions/checkout@v2 | ||
|
||
- name: Set up Python ${{ matrix.python-version }} | ||
uses: actions/setup-python@v2 | ||
with: | ||
python-version: ${{ matrix.python-version }} | ||
architecture: ${{ matrix.architecture }} | ||
|
||
- name: flake8 check | ||
run: | | ||
python -m pip install -q --upgrade pip | ||
python -m pip install -q flake8 | ||
flake8 | ||
- name: Validate changed URLs from this PR | ||
run: | | ||
git fetch --all | ||
python workflow_scripts/validate_changed_urls.py | ||
- name: Weekly validate all URLs under onnx/tutorials | ||
if: (github.event_name == 'schedule') # Only triggered by weekly event | ||
run: | | ||
python workflow_scripts/validate_all_urls.py |
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 |
---|---|---|
|
@@ -4,3 +4,4 @@ | |
.DS_Store | ||
.idea/ | ||
!/tutorials/output/README.md | ||
*.pyc |
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
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
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
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
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
2 changes: 1 addition & 1 deletion
2
...eML/ONNXLive/ONNXLive.xcodeproj/project.xcworkspace/xcshareddata/IDEWorkspaceChecks.plist
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
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
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
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 @@ | ||
[flake8] | ||
max-line-length = 130 |
Oops, something went wrong.