Skip to content
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

feature/generic update #82

Merged
merged 3 commits into from
Dec 15, 2023
Merged

feature/generic update #82

merged 3 commits into from
Dec 15, 2023

Conversation

nicorikken
Copy link
Member

  • feat: rework GitHub Actions setup
  • feat: update locked poetry dependencies
  • feat: ci-build workflow to at least build on pull-requests

Update to setup-python v4 and use recommendations from the docs
https://github.com/actions/setup-python/blob/main/docs/advanced-usage.md

Signed-off-by: Nico Rikken <nico.rikken@alliander.com>
Signed-off-by: Nico Rikken <nico.rikken@alliander.com>
To ensure that the building of documentation has been tested.

Also improve the poetry install method.

Signed-off-by: Nico Rikken <nico.rikken@alliander.com>
@nicorikken nicorikken force-pushed the feature/generic-update branch from 6f75baf to cf98e50 Compare November 9, 2023 10:03
@nicorikken
Copy link
Member Author

@marcvanandel hierbij een poging om het bouwproces vlot te trekken en te verbeteren voor de toekomst.

Copy link
Member

@marcvanandel marcvanandel left a comment

Choose a reason for hiding this comment

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

great changes! Looks good to me ... and we can only test my merging, right? Let's go!

Thanks for your great work, Nico!

@marcvanandel marcvanandel merged commit 7dba5fc into main Dec 15, 2023
1 check passed
@marcvanandel marcvanandel deleted the feature/generic-update branch December 15, 2023 11:45
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