Skip to content

Update dependency (SA-674)#22

Open
dstewartons wants to merge 3 commits intomainfrom
SA-674/soc-lookup-csv-compatibilty
Open

Update dependency (SA-674)#22
dstewartons wants to merge 3 commits intomainfrom
SA-674/soc-lookup-csv-compatibilty

Conversation

@dstewartons
Copy link
Copy Markdown
Contributor

@dstewartons dstewartons commented Apr 30, 2026

📌 Pull Request Template

Please complete all sections

✨ Summary

Single dependency bump: survey-assist-utils v0.0.7v0.0.8, with an updated poetry.lock.

📜 Changes Introduced

  • pyproject.tomlsurvey-assist-utils tag v0.0.8
  • poetry.lock — regenerated

Commits: 23814c3, 8b5f207, d950f18

  • Feature implementation (feat:) / bug fix (fix:) / refactoring (chore:) / documentation (docs:) / testing (test:)
  • Updates to tests and/or documentation (not required for this dependency-only change)
  • Terraform changes (if applicable) - N/A (no Terraform changes in this repo)

✅ Checklist

Please confirm you've completed these checks before requesting a review.

  • Code is formatted using Black
  • Imports are sorted using isort
  • Code passes linting with Ruff, Pylint, and Mypy
  • Security checks pass using Bandit
  • API and Unit tests are written and pass using pytest
  • Terraform files (if applicable) follow best practices and have been validated (terraform fmt & terraform validate) - N/A
  • DocStrings follow Google-style and are added as per Pylint recommendations
  • Documentation has been updated if needed - N/A for this PR

🔍 How to Test

From soc-classification-utils repo root:

  1. Install dependencies

    poetry install
  2. Run quality and security checks

    make check-python
  3. Run the test suite

    poetry run pytest

If poetry lock fails with 'Tag' object has no attribute 'parents'

Stale Git clones under $(poetry env info -p)/src/. For example:

rm -rf "$(poetry env info -p)/src/soc-classification-library" "$(poetry env info -p)/src/survey-assist-utils"
poetry lock && poetry install

@dstewartons dstewartons changed the title update dependency Update dependency (SA-674) May 1, 2026
Copy link
Copy Markdown

@Tom-Owen-ONS Tom-Owen-ONS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues here, looks good to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants