Skip to content

Conversation

@teutoburg
Copy link
Contributor

This should now really actually (I hope) solve the recent macOS issues, by installing Poetry first and then the correct Python version, as suggested by the actions/setup-python docs. I also now got rid of the abatilo/actions-poetry, that project appears abandoned by the (only) author, and after looking at the actual code, it does little more than running pipx install poetry anyway.

Looking at the previously used action workflow's source code
(https://github.com/abatilo/actions-poetry/blob/master/action.yml),
this seems to be all it does anyway. So probably better to just do it
manually and have full control, if that's all we need.

This will most likely not solve the macOS issue, but at least it should
make it easier to debug and fix it...
Running from Poetry to avoid any environment crape.
@codecov
Copy link

codecov bot commented Aug 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (a449ba9) to head (222c826).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #56   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            2         2           
  Lines            7         7           
=========================================
  Hits             7         7           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@teutoburg teutoburg marked this pull request as ready for review August 22, 2025 09:54
@teutoburg teutoburg moved this to 👀 Awaiting Review in ScopeSim-development Aug 22, 2025
@teutoburg teutoburg requested a review from a team August 22, 2025 09:55
Copy link
Contributor

@hugobuddel hugobuddel left a comment

Choose a reason for hiding this comment

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

Good! Thanks.

(And now let's ditch everything in favor of uv?)

@teutoburg
Copy link
Contributor Author

(And now let's ditch everything in favor of uv?)

Nah, please don't. Already now while googling for solutions to this, half the results were quasi-advertisements for uv...

@teutoburg teutoburg merged commit 163b900 into main Aug 24, 2025
20 checks passed
@teutoburg teutoburg deleted the fh/manualpoetry branch August 24, 2025 11:42
@github-project-automation github-project-automation bot moved this from 👀 Awaiting Review to ✅ Done in ScopeSim-development Aug 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants