Skip to content

ci: migrate to reusable workflows - #328

Open
CodeVishal-17 wants to merge 1 commit into
openclimatefix:mainfrom
CodeVishal-17:ci/use-reusable-workflows
Open

ci: migrate to reusable workflows#328
CodeVishal-17 wants to merge 1 commit into
openclimatefix:mainfrom
CodeVishal-17:ci/use-reusable-workflows

Conversation

@CodeVishal-17

@CodeVishal-17 CodeVishal-17 commented Jan 7, 2026

Copy link
Copy Markdown
Contributor

Description

Migrates the repository to use OpenClimateFix reusable GitHub Actions workflows.
Legacy branch and PR workflows have been disabled to avoid duplicate CI runs, while preserving scheduled jobs.
This aligns the repo with the new organisation-wide CI setup and ensures HF_TOKEN is passed correctly where required.

Related to openclimatefix/.github#86

How Has This Been Tested?

This change updates CI configuration only.
Testing is handled by GitHub Actions: the updated reusable workflows will run automatically on this PR and verify the setup.

  • Yes

If your changes affect data processing, have you plotted any changes? i.e. have you done a quick sanity check?

  • Yes

Checklist:

  • My code follows OCF's coding style guidelines
  • I have performed a self-review of my own code
  • I have checked my code and corrected any misspellings

@CodeVishal-17

Copy link
Copy Markdown
Contributor Author

Quick clarification on the scope of this PR:

This PR is primarily focused on migrating open-source-quartz-solar-forecast to the new OpenClimateFix reusable GitHub Actions workflows, in line with the org-wide CI changes.

During CI runs, we uncovered an existing issue where Sentry’s HuggingFaceHub integration fails during pytest collection (InferenceClient.chat_completion), which appears to be related to a dependency compatibility mismatch rather than the workflow migration itself.

I attempted a few minimal mitigations (skipping Sentry initialisation in CI/pytest) to unblock tests, but since this behaviour was not part of the original request and may need a more deliberate fix, I’m happy to revert those changes and keep this PR strictly scoped to CI migration only.

Please let me know how you’d prefer to proceed:

keep this PR limited to reusable workflow migration, or

handle the Sentry/HuggingFace issue separately in a follow-up PR.

Happy to adjust accordingly.

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.

1 participant