-
Notifications
You must be signed in to change notification settings - Fork 2
1.8.3 #221
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
1.8.3 #221
Changes from all commits
Commits
Show all changes
50 commits
Select commit
Hold shift + click to select a range
f8af791
updating version numbers
digitalghost-dev 0618cf7
accidental commit
digitalghost-dev fd9ce70
adding paths to ignore
digitalghost-dev 9e9f791
updating golden test
digitalghost-dev 485bd18
adding type hints
digitalghost-dev bff8360
updating schedule
digitalghost-dev f98bf95
adding type hints
digitalghost-dev 5feeb5a
returning a `struct` instead of separate values (#218)
digitalghost-dev b24d076
removing redundant line, using `pf.FlagSet.Usage` instead (#218)
digitalghost-dev 1c9f0d1
updating `checkout` action to `v6`
digitalghost-dev caff2ea
returning a `struct` instead of separate values (#218)
digitalghost-dev bf2a5b7
expanding list comprehension for better legibility
digitalghost-dev 58e2ee9
applying `ruff` formatting
digitalghost-dev 03ad041
sorting imports
digitalghost-dev 8720b23
removing type hint
digitalghost-dev 46ff724
update comments
digitalghost-dev dab43ea
refactoring for better testability (#220)
digitalghost-dev 4693545
adding pyrefly and bandit tooling
digitalghost-dev d6f1091
adding project layout section
digitalghost-dev a229903
updating docs
digitalghost-dev 38d75fa
adding config file for `gitleaks`
digitalghost-dev 472cc47
adding config file for `gitleaks`
digitalghost-dev d44d711
adding gitleaks ci job
digitalghost-dev 99bd65d
adding comments for bandit security
digitalghost-dev a868017
adding helper function for calling the APIs
digitalghost-dev c75b68c
updating directory layout
digitalghost-dev f9fa33c
updating import paths
digitalghost-dev 869c8cb
updating directory layout
digitalghost-dev 5f27b48
adding data quality checks for sets table
digitalghost-dev 854cc0a
filtering out special sets
digitalghost-dev f3372ac
updating reference name
digitalghost-dev 2a352e9
adding in soda checks
digitalghost-dev fb7d9a9
updating function names
digitalghost-dev 6a9663c
removing `os.Exit()` from commands, handling in `main()` (#219)
digitalghost-dev f725bc2
updating test coverage
digitalghost-dev 943ee0c
updating test coverage
digitalghost-dev 74c0c47
updating main repo photo
digitalghost-dev 5f08eae
fixing linter issues
digitalghost-dev 5314f9b
setting false to fail on error
digitalghost-dev 7de5c8e
removing unused import
digitalghost-dev 39a198c
updating function name
digitalghost-dev 7848e99
increasing test coverage
digitalghost-dev 3a90ede
updating project layout section
digitalghost-dev 6bd3033
fixing typo
digitalghost-dev 7bd34b3
updating test coverage
digitalghost-dev 9195c88
small edits
digitalghost-dev f22f779
initial codspeed action commit
digitalghost-dev 906395a
updating go version
digitalghost-dev 68b80ee
adding paths to ignore
digitalghost-dev 3940e1a
adding codspeed benchmarking
digitalghost-dev 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,39 @@ | ||
| name: Codspeed Benchmarks | ||
| on: | ||
| pull_request: | ||
| types: [ opened, reopened, synchronize ] | ||
| paths: | ||
| - 'card_data/**' | ||
|
|
||
| permissions: | ||
| contents: read | ||
| id-token: write | ||
|
|
||
| jobs: | ||
| benchmarks: | ||
| runs-on: ubuntu-22.04 | ||
| defaults: | ||
| run: | ||
| working-directory: card_data | ||
|
|
||
| steps: | ||
| - name: Checkout | ||
| uses: actions/checkout@v6 | ||
|
|
||
| - name: Setup Python | ||
| uses: actions/setup-python@v6 | ||
| with: | ||
| python-version: '3.12' | ||
|
|
||
| - name: Install uv | ||
| uses: astral-sh/setup-uv@v4 | ||
|
|
||
| - name: Install dependencies | ||
| run: uv sync --dev | ||
|
|
||
| - name: Run the benchmarks | ||
| uses: CodSpeedHQ/action@v4 | ||
| with: | ||
| working-directory: card_data | ||
| mode: simulation | ||
| run: uv run pytest pipelines/tests/ -v --codspeed |
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 |
|---|---|---|
|
|
@@ -66,3 +66,5 @@ card_data/~/ | |
| card_data/storage/ | ||
| /.claude/ | ||
| CLAUDE.md | ||
| REFACTORING.md | ||
| /card_data/.codspeed/ | ||
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 |
|---|---|---|
| @@ -1,3 +1,3 @@ | ||
| cmd/card/setslist.go:generic-api-key:116 | ||
| cmd/card/cardlist.go:generic-api-key:157 | ||
| codecov.yml:generic-api-key:2 | ||
| codecov.yml:generic-api-key:2 | ||
| card_data/sample_scripts/ebay.py:generic-api-key:6 | ||
| card_data/sample_scripts/get_data.py:generic-api-key:9 | ||
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
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.