-
Notifications
You must be signed in to change notification settings - Fork 34
chore: release for chain v1.20.3 #421
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -14,7 +14,7 @@ jobs: | |
| - name: Install Python | ||
| uses: actions/setup-python@v5 | ||
| - name: Install poetry | ||
| run: python -m pip install poetry | ||
| run: python -m pip install "poetry>=2.4,<3" | ||
|
Comment on lines
14
to
+17
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🎯 Functional Correctness | 🟠 Major | ⚡ Quick win 🧩 Analysis chain🏁 Script executed: #!/bin/bash
set -euo pipefail
echo "=== release workflow excerpt ==="
sed -n '1,80p' .github/workflows/release.yml
echo
echo "=== python version declarations ==="
rg -n --hidden --glob '!**/.git/**' -e 'python-version|requires-python|tool\.poetry\.dependencies|Programming Language :: Python ::' pyproject.toml poetry.lock .github/workflows README* .python-version .tool-versions setup.cfg setup.py 2>/dev/null || true
echo
echo "=== candidate config files ==="
fd -H -a 'pyproject.toml|poetry.lock|.python-version|.tool-versions|setup.cfg|setup.py' .Repository: InjectiveLabs/sdk-python Length of output: 5335 🏁 Script executed: #!/bin/bash
set -euo pipefail
echo "=== pyproject.toml excerpt ==="
sed -n '1,120p' pyproject.toml
echo
echo "=== workflow files using setup-python ==="
rg -n "actions/setup-python@|python-version:" .github/workflowsRepository: InjectiveLabs/sdk-python Length of output: 2904 🌐 Web query:
💡 Result: In actions/setup-python v5, the python-version input is optional [1][2]. If this input is omitted, the action follows a specific fallback behavior to determine which Python version to use: 1. File-based resolution: The action first attempts to read the version from a.python-version file in the repository root [1][3]. 2. PATH fallback: If no.python-version file exists, the action will use the Python or PyPy version currently available in the PATH [1][4]. Because the default version present in the PATH can vary between different runner images and may change unexpectedly over time, the official documentation strongly recommends that you always explicitly specify a Python version using the python-version or python-version-file inputs to ensure build consistency [1][5]. Citations:
Pin the release Python version. 🤖 Prompt for AI Agents |
||
| - name: Publish package | ||
| env: | ||
| PYPI_TOKEN: ${{ secrets.PYPI_API_TOKEN }} | ||
|
|
||
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -765,6 +765,10 @@ async def test_fetch_positions_v2( | |
| "cumulativeFundingEntry": position.cumulative_funding_entry, | ||
| "effectiveCumulativeFundingEntry": position.effective_cumulative_funding_entry, | ||
| "upnl": position.upnl, | ||
| "initialLeverage": position.initial_leverage, | ||
| "initialMargin": position.initial_margin, | ||
| "initialEntryPrice": position.initial_entry_price, | ||
| "initialQuantity": position.initial_quantity, | ||
|
Comment on lines
+768
to
+771
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🗄️ Data Integrity & Integration | 🟡 Minor | ⚡ Quick win Use non-default values for the new position fields. Both tests currently assert protobuf defaults, so they do not reliably detect incorrect propagation or serialization.
📍 Affects 2 files
🤖 Prompt for AI Agents |
||
| }, | ||
| ], | ||
| "paging": { | ||
|
|
||
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.
🩺 Stability & Availability | 🟠 Major | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
Repository: InjectiveLabs/sdk-python
Length of output: 1314
🏁 Script executed:
Repository: InjectiveLabs/sdk-python
Length of output: 1314
🏁 Script executed:
Repository: InjectiveLabs/sdk-python
Length of output: 1089
Include the resolved Python version in the
.venvcache key. The current key is shared across all versions that satisfy>=3.9 <3.12, so a cache built under one interpreter can be restored under another. Add the resolvedsetup-pythonversion to the key, or pin the workflow to an exact version.🤖 Prompt for AI Agents