Skip to content

Gh pages update#795

Merged
calvinp0 merged 3 commits intomainfrom
gh-pages-update
Jul 29, 2025
Merged

Gh pages update#795
calvinp0 merged 3 commits intomainfrom
gh-pages-update

Conversation

@calvinp0
Copy link
Copy Markdown
Member

No description provided.

@calvinp0 calvinp0 requested a review from Copilot July 29, 2025 07:54

This comment was marked as outdated.

calvinp0 added 3 commits July 29, 2025 13:56
Updates the pydas dependency to the latest version.
This ensures compatibility and access to the newest features and bug fixes.
Adds pull request triggers to the cache update workflow,
ensuring the cache is up-to-date on pull requests.

Changes the environment update command from `mamba` to `conda` to ensure compatibility.

Only caches the ARC environment on push events,
preventing unnecessary cache updates on other events.
Configures a GitHub Actions workflow to build and deploy documentation to GitHub Pages.

This workflow utilizes Micromamba to manage isolated environments for building RMG-Py and ARC, ensuring consistent and reproducible builds. It also installs necessary system dependencies for documentation generation (TeX).

The workflow builds the HTML documentation and deploys it to the `gh-pages` branch only on `push` events to the `main` or `docs` branches.
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 updates the GitHub Pages workflow to use micromamba instead of conda/miniconda, adds pull request triggers for workflow testing, and updates various dependencies and configurations.

Key changes:

  • Replaces conda/miniconda setup with micromamba for better performance and caching
  • Adds pull request triggers to both GitHub Pages and cache update workflows
  • Updates pydas dependency version from >=1.0.2 to >=1.0.3

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
environment.yml Updates pydas dependency version requirement
.github/workflows/update-cache.yml Adds PR triggers and switches from mamba to conda for environment updates
.github/workflows/gh-pages.yml Major refactor to use micromamba, adds PR triggers, and improves documentation build process

Comment thread .github/workflows/update-cache.yml
Comment thread .github/workflows/gh-pages.yml
Copy link
Copy Markdown
Collaborator

@kfir4444 kfir4444 left a comment

Choose a reason for hiding this comment

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

lgtm

@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.49%. Comparing base (80621b7) to head (1054aab).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #795   +/-   ##
=======================================
  Coverage   74.49%   74.49%           
=======================================
  Files         103      103           
  Lines       28121    28121           
  Branches     5771     5771           
=======================================
  Hits        20950    20950           
  Misses       5680     5680           
  Partials     1491     1491           
Flag Coverage Δ
unittests 74.49% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

@calvinp0 calvinp0 merged commit 9ba9803 into main Jul 29, 2025
12 of 13 checks passed
@calvinp0 calvinp0 deleted the gh-pages-update branch July 29, 2025 14:29
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.

3 participants