-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#418: Introducing python toolbox #421
Merged
Merged
Changes from 20 commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
7151f59
#418: Introduced Python-Toolbox
tomuben a8e59a8
Updated changelog
tomuben 4e9eb87
Set pylint minimal rating
tomuben 3b13ae8
Updated .gitignore
tomuben 90d7600
Replace usages of Python3.8 with Ubuntu22.04+Python3.10 or Python3.9
tomuben 6d446cd
Marked automatically generated GH workflows as such
tomuben 3d20027
Update .pre-commit-config.yaml
tomuben 043e573
Updated .pre-commit-config.yaml
tomuben e84ef05
Updated doc
tomuben d8c1f5b
Ignore check in checks.yml
tomuben 30a0078
[run all tests]
tomuben f786764
Updated unreleased.md
tomuben a4afc19
Updated developer_guide.rst
tomuben 21926e9
[run all tests]
tomuben f170c54
Updated release steps in developer guide
tomuben b3fe249
Removed some GH workflows from .gitattributes
tomuben e74fc13
Use matrix build in test_python_version.yml
tomuben aa6ca55
Restructed GH workflows for tests
tomuben 2fc67cb
Removed all GH Workflows from .gitattributes
tomuben fad1093
Fixed security scanner
tomuben 7fddee8
[run all tests]
tomuben 67d9cab
[run all tests]
tomuben 577f86b
1. Override unit test nox task
tomuben e898cf0
[run all tests]
tomuben 2c8e28e
Cleaned up .pre-commit-config.yaml
tomuben 3bc328c
[run all tests]
tomuben 796cd6d
Added empty webfonts.html
tomuben 0440ab7
Renamed tests.yml to tests_with_converage.yml
tomuben 0776b0e
[run all tests]
tomuben c35769b
Renamed tests_with_converage.yml to tests_with_coverage.yml and fixe…
tomuben 24ed63c
[run all tests]
tomuben 950c0b4
Updated to exasol-toolbox 0.18.0 and removed overloading nox task lin…
tomuben 72b1c2d
[run all tests]
tomuben File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
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: CD | ||
|
||
on: | ||
push: | ||
tags: | ||
- '**' | ||
|
||
jobs: | ||
|
||
check-tag-version-job: | ||
name: Check Release Tag | ||
uses: ./.github/workflows/check-release-tag.yml | ||
|
||
cd-job: | ||
name: Continuous Delivery | ||
uses: ./.github/workflows/build-and-publish.yml | ||
secrets: | ||
PYPI_TOKEN: ${{ secrets.PYPI_TOKEN }} | ||
|
||
publish-docs: | ||
needs: [ cd-job ] | ||
name: Publish Documentation | ||
uses: ./.github/workflows/gh-pages.yml | ||
|
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,21 @@ | ||
name: Check Release Tag | ||
|
||
on: workflow_call | ||
|
||
jobs: | ||
|
||
check-tag-version-job: | ||
|
||
name: Check Tag Version | ||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- name: SCM Checkout | ||
uses: actions/checkout@v4 | ||
|
||
- name: Setup Python & Poetry Environment | ||
uses: exasol/python-toolbox/.github/actions/[email protected] | ||
|
||
- name: Check Tag Version | ||
# make sure the pushed/created tag matched the project version | ||
run: "[[ `poetry version --short` == ${{ github.ref_name }} ]]" |
This file was deleted.
Oops, something went wrong.
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,123 @@ | ||
name: Checks | ||
|
||
on: | ||
workflow_call: | ||
secrets: | ||
ALTERNATIVE_GITHUB_TOKEN: | ||
required: false | ||
|
||
jobs: | ||
|
||
Version-Check: | ||
name: Version | ||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- name: SCM Checkout | ||
uses: actions/checkout@v4 | ||
with: | ||
fetch-depth: 0 | ||
|
||
- name: Setup Python & Poetry Environment | ||
uses: exasol/python-toolbox/.github/actions/[email protected] | ||
|
||
- name: Check Version(s) | ||
run: | | ||
poetry run version-check `poetry run python -c "from noxconfig import PROJECT_CONFIG; print(PROJECT_CONFIG.version_file)"` | ||
|
||
Documentation: | ||
name: Docs | ||
needs: [ Version-Check ] | ||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- name: SCM Checkout | ||
uses: actions/checkout@v4 | ||
|
||
- name: Setup Python & Poetry Environment | ||
uses: exasol/python-toolbox/.github/actions/[email protected] | ||
|
||
- name: Build Documentation | ||
run: | | ||
poetry run python -m nox -s docs:build | ||
|
||
Lint: | ||
name: Linting (Python-${{ matrix.python-version }}) | ||
needs: [ Version-Check ] | ||
runs-on: ubuntu-latest | ||
strategy: | ||
fail-fast: false | ||
matrix: | ||
python-version: ["3.9", "3.10", "3.11"] | ||
|
||
steps: | ||
- name: SCM Checkout | ||
uses: actions/checkout@v4 | ||
|
||
- name: Setup Python & Poetry Environment | ||
uses: exasol/python-toolbox/.github/actions/[email protected] | ||
with: | ||
python-version: ${{ matrix.python-version }} | ||
|
||
- name: Run lint | ||
run: poetry run nox -s lint:code | ||
|
||
- name: Upload Artifacts | ||
uses: actions/[email protected] | ||
with: | ||
name: lint-python${{ matrix.python-version }} | ||
path: .lint.txt | ||
include-hidden-files: true | ||
|
||
Type-Check: | ||
name: Type Checking (Python-${{ matrix.python-version }}) | ||
needs: [ Version-Check ] | ||
runs-on: ubuntu-latest | ||
strategy: | ||
fail-fast: false | ||
matrix: | ||
python-version: ["3.9", "3.10", "3.11"] | ||
|
||
steps: | ||
- name: SCM Checkout | ||
uses: actions/checkout@v4 | ||
|
||
- name: Setup Python & Poetry Environment | ||
uses: exasol/python-toolbox/.github/actions/[email protected] | ||
with: | ||
python-version: ${{ matrix.python-version }} | ||
|
||
- name: Run type-check | ||
run: poetry run nox -s lint:typing || echo ignoring... | ||
|
||
Security: | ||
name: Security Checks (Python-${{ matrix.python-version }}) | ||
needs: [ Version-Check ] | ||
runs-on: ubuntu-latest | ||
strategy: | ||
fail-fast: false | ||
matrix: | ||
python-version: [ "3.9", "3.10", "3.11" ] | ||
|
||
steps: | ||
- name: SCM Checkout | ||
uses: actions/checkout@v4 | ||
|
||
- name: Setup Python & Poetry Environment | ||
uses: exasol/python-toolbox/.github/actions/[email protected] | ||
with: | ||
python-version: ${{ matrix.python-version }} | ||
|
||
- name: Run security linter | ||
run: poetry run nox -s lint:security || echo ignoring... | ||
|
||
- name: Upload Artifacts | ||
uses: actions/[email protected] | ||
with: | ||
name: security-python${{ matrix.python-version }} | ||
path: .security.json | ||
include-hidden-files: true | ||
|
||
Tests: | ||
name: Tests | ||
uses: ./.github/workflows/tests.yml |
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: CI | ||
|
||
on: | ||
push: | ||
branches-ignore: | ||
- "github-pages/*" | ||
- "gh-pages/*" | ||
- "main" | ||
- "master" | ||
pull_request: | ||
types: [opened, reopened] | ||
schedule: | ||
# “At 00:00 on every 7th day-of-month from 1 through 31.” (https://crontab.guru) | ||
- cron: "0 0 1/7 * *" | ||
|
||
jobs: | ||
|
||
CI: | ||
uses: ./.github/workflows/merge-gate.yml | ||
secrets: inherit | ||
|
||
Metrics: | ||
needs: [ CI ] | ||
uses: ./.github/workflows/report.yml |
This file was deleted.
Oops, something went wrong.
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 |
---|---|---|
|
@@ -8,8 +8,13 @@ jobs: | |
get_exasol_versions: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v3 | ||
- uses: ./.github/actions/prepare_poetry_env | ||
- uses: actions/checkout@v4 | ||
- name: Setup Python & Poetry Environment | ||
uses: exasol/python-toolbox/.github/actions/[email protected] | ||
with: | ||
python-version: "3.10" | ||
poetry-version: '1.8.2' | ||
|
||
- name: Print matrix | ||
run: poetry run nox --non-interactive -s "get-all-db-versions" | ||
- id: set-matrix | ||
|
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: Publish Documentation | ||
|
||
on: | ||
workflow_call: | ||
workflow_dispatch: | ||
|
||
jobs: | ||
|
||
documentation-job: | ||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- name: SCM Checkout | ||
uses: actions/checkout@v4 | ||
with: | ||
fetch-depth: 0 | ||
|
||
- name: Setup Python & Poetry Environment | ||
uses: exasol/python-toolbox/.github/actions/[email protected] | ||
|
||
- name: Build Documentation | ||
run: | | ||
poetry run sphinx-multiversion doc/ .html-documentation | ||
|
||
- name: Deploy | ||
uses: JamesIves/[email protected] | ||
with: | ||
branch: gh-pages | ||
folder: .html-documentation | ||
git-config-name: Github Action | ||
git-config-email: [email protected] |
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,26 @@ | ||
name: Merge-Gate | ||
|
||
on: | ||
workflow_call: | ||
secrets: | ||
ALTERNATIVE_GITHUB_TOKEN: | ||
required: false | ||
|
||
jobs: | ||
|
||
checks: | ||
name: Checks | ||
uses: ./.github/workflows/checks.yml | ||
|
||
# This job ensures inputs have been executed successfully. | ||
approve-merge: | ||
name: Allow Merge | ||
runs-on: ubuntu-latest | ||
# If you need additional jobs to be part of the merge gate, add them below | ||
needs: [ checks ] | ||
|
||
# Each job requires a step, so we added this dummy step. | ||
steps: | ||
- name: Approve | ||
run: | | ||
echo "Merge Approved" |
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,25 @@ | ||
name: PR-Merge | ||
|
||
on: | ||
push: | ||
branches: | ||
- 'main' | ||
- 'master' | ||
|
||
jobs: | ||
|
||
# This job can be removed if certain preconditions are met. See | ||
# https://exasol.github.io/python-toolbox/user_guide/workflows.html#pr-merge-workflow | ||
|
||
ci-job: | ||
name: Checks | ||
uses: ./.github/workflows/checks.yml | ||
secrets: inherit | ||
|
||
publish-docs: | ||
name: Publish Documentation | ||
uses: ./.github/workflows/gh-pages.yml | ||
|
||
metrics: | ||
needs: [ ci-job ] | ||
uses: ./.github/workflows/report.yml |
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
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I personally would remove this job