Skip to content

Add CI for technologydata prototype#10

Merged
finozzifa merged 11 commits intoprototypefrom
feat/add_ci_prototype
Jun 26, 2025
Merged

Add CI for technologydata prototype#10
finozzifa merged 11 commits intoprototypefrom
feat/add_ci_prototype

Conversation

@finozzifa
Copy link
Copy Markdown
Member

@finozzifa finozzifa commented Jun 25, 2025

Closes # (if applicable).

Changes proposed in this Pull Request

Checklist

  • Code changes are sufficiently documented; i.e. new functions contain docstrings and further explanations may be given in doc.
  • Data source for new technologies is clearly stated.
  • Newly introduced dependencies are added to environment.yaml (if applicable).
  • A note for the release notes doc/release_notes.rst of the upcoming release is included.
  • I consent to the release of this PR's code under the GPLv3 license.

@finozzifa finozzifa requested a review from euronion June 25, 2025 14:08
@finozzifa finozzifa marked this pull request as ready for review June 25, 2025 14:08
@euronion euronion requested a review from Copilot June 25, 2025 15:45
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a Continuous Integration workflow for the technologydata prototype.

  • Adds a new GitHub Actions workflow for testing on the prototype branch.
  • Configures job steps to check out the code, set up Python 3.13, install dependencies via uv, and run unit tests.

Comment thread .github/workflows/ci.yaml
Comment thread .github/workflows/ci.yaml
@euronion
Copy link
Copy Markdown
Member

Thanks!

I've added a cache, reducing CI run times by ~2 minutes.
My guess is this is related to pydeflate being slow in reading, i.e. each call to pydeflate takes a bit of time.

Might be a potential performance improvement at some point.

PR LGTM, can merge 👍

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@finozzifa finozzifa merged commit 03abe0b into prototype Jun 26, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants