-
Notifications
You must be signed in to change notification settings - Fork 105
feat(core): restore default task opts, improve workflows #560
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
Merged
Merged
Changes from all commits
Commits
Show all changes
49 commits
Select commit
Hold shift + click to select a range
41a6976
feat(tasks): add missing task options and restore defaults to tools d…
ocervell 7bffa56
Merge branch 'main' into improve-defaults-tools-opts
ocervell 062b796
Merge branch 'main' into improve-defaults-tools-opts
ocervell a64a81b
Merge branch 'main' into improve-defaults-tools-opts
ocervell f9ad885
remove comment
ocervell 1bf5eeb
update
ocervell 07ae095
rework inputs filtering to revert to inputs
ocervell 3614f2c
remove -Pn from default opts
ocervell 800b5eb
update
ocervell 75f5626
update
ocervell 86cb9af
update
ocervell d47acec
update
ocervell 7420e03
pin tools versions, add workflow for checking tasks, fix integration …
ocervell ca1d5dc
fix dirsearch json option with latest version
ocervell 6c4d352
improve workflow
ocervell 0a03212
update workflow
ocervell a24fea6
8000
update workflow
ocervell 129136e
update
ocervell 7226f11
update
ocervell 18a8451
Merge branch 'main' into improve-defaults-tools-opts
ocervell b8c8967
update
ocervell 5418466
update
ocervell 502ee29
update
ocervell 7bb5343
update
ocervell 8e6a1e0
update
ocervell 2830874
update
ocervell 7aeeca2
updatte
ocervell 46bb48f
update
ocervell 574239a
updatre
ocervell f049454
update
ocervell a8ed357
update
ocervell a80f1c0
update
ocervell 68c65a0
update
ocervell a100264
update
ocervell 34b64ec
update
ocervell 15bcd26
update
ocervell 8b98c48
update
ocervell bd7c4c3
update
ocervell e2f0c70
update
ocervell 66c7898
update
ocervell 3e8ba35
update
ocervell 01c6934
update
ocervell d8962c2
update
ocervell e71a44e
update
ocervell 15ecc2b
update
ocervell 47f1257
update
ocervell d13112c
use cls.cmd to infer command name to fetch version
ocervell 8d8cfb6
update
ocervell cb88073
update
ocervell 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 contains hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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,56 @@ | ||
name: Tasks | ||
|
||
on: | ||
push: | ||
paths: | ||
- 'secator/tasks/*.py' | ||
branches: | ||
- release-please* | ||
pull_request: | ||
types: [labeled, synchronize] | ||
paths: | ||
- 'secator/tasks/*.py' | ||
|
||
concurrency: | ||
group: ${{ github.workflow }}-${{ github.event_name }}-${{ github.event.pull_request.number || github.ref }} | ||
cancel-in-progress: true | ||
|
||
jobs: | ||
test-tasks: | ||
runs-on: ${{ matrix.os }} | ||
strategy: | ||
fail-fast: false | ||
matrix: | ||
# os: ["ubuntu-latest", "macos-latest"] | ||
os: ["ubuntu-latest"] | ||
steps: | ||
- name: Checkout repository | ||
uses: actions/checkout@v3 | ||
with: | ||
fetch-depth: 0 | ||
|
||
- name: Install secator | ||
uses: ./.github/actions/install | ||
with: | ||
python-version: ${{ matrix.python-version }} | ||
|
||
- name: Detect changed or all tasks | ||
run: | | ||
if [[ ${{ github.event_name }} == "push" && ${{ github.ref_name }} =~ ^release-please.* ]]; then | ||
TASKS=$(secator x --list) | ||
echo "Running all tasks" | ||
else | ||
TASKS=$(git diff --name-only --diff-filter=AMR origin/main HEAD | grep '^secator/tasks/.*\.py$' | grep -v '_.*\.py$' | sed 's/\.py$//' | sed 's/^secator\/tasks\///') | ||
echo "Changed tasks: $TASKS" | ||
fi | ||
|
||
FAILED=0 | ||
for task in $TASKS; do | ||
if ! secator test task -c $task; then | ||
FAILED=1 | ||
fi | ||
echo "" | ||
done | ||
if [ $FAILED -eq 1 ]; then | ||
exit 1 | ||
fi | ||
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.
Uh oh!
There was an error while loading. Please reload this page.