chore: add dependency lock files#214
Merged
Merged
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #214 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 9 9
Lines 1604 1604
Branches 175 175
=========================================
Hits 1604 1604
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
Update facebook_post and release_setup Dockerfiles to set PYTHONDONTWRITEBYTECODE and run uv sync inside a heredoc. Create an unprivileged 'action' group/user and switch to USER action to follow least-privilege practices and avoid writing .pyc files in the container.
ReenigneArcher
force-pushed
the
chore/add-dependency-lock-files
branch
2 times, most recently
from
a29b27a to
889a5ef
Compare
Make CI use the action-provided Python and uv runner: add an install_uv helper and logic in actions/setup_python/post-ci.sh to locate ACTION_PYTHON_PATH, install uv, and run tests with `uv run --project ... --frozen --extra dev --python 3.14`. Update pyproject.toml to require Python >=3.14 and pin pytest to 9.0.3. Add a uv.lock file. Update tests to use a new action_python_path fixture and helper to invoke the action's interpreter, adjusting tests to call the external Python rather than relying on the test process's interpreter.
ReenigneArcher
force-pushed
the
chore/add-dependency-lock-files
branch
from
889a5ef to
4cbd07b
Compare
Add NOSONAR comment and remove the PYTHONDONTWRITEBYTECODE env and the creation/switch to an unprivileged "action" user. Run `uv sync --frozen --no-dev --no-install-project` as root so GitHub Actions can write to mounted workspace paths. Changes applied to actions/facebook_post and actions/release_setup Dockerfiles.
ReenigneArcher
force-pushed
the
chore/add-dependency-lock-files
branch
from
176dcc9 to
84af339
Compare
Remove reliance on ACTION_PYTHON_PATH in CI and make tests handle a missing action-provided interpreter. The workflow no longer sets ACTION_PYTHON_PATH; tests now use os.environ.get to avoid KeyError, return in-process platform/python version when ACTION_PYTHON_PATH is absent, and consolidate inline scripts into PLATFORM_VERSION_SCRIPT and SYSTEM_VERSION_SCRIPT constants. This allows the test suite to run with the current Python interpreter if the action path isn't provided.
ReenigneArcher
force-pushed
the
chore/add-dependency-lock-files
branch
from
84af339 to
8b27d12
Compare
|
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Description
Screenshot
Issues Fixed or Closed
Roadmap Issues
Type of Change
Checklist
AI Usage