Skip to content

[CLEANUP] Update workflows and environment#69

Merged
eleanorfrajka merged 4 commits into
mainfrom
eleanor-workflows-1
Sep 20, 2025
Merged

[CLEANUP] Update workflows and environment#69
eleanorfrajka merged 4 commits into
mainfrom
eleanor-workflows-1

Conversation

@eleanorfrajka

@eleanorfrajka eleanorfrajka commented Sep 20, 2025

Copy link
Copy Markdown
Collaborator

Description:

Updated workflows to use micromamba (with environment.yml) instead of pip (with requirements-dev.txt).

Added environment.yml for conda usage.

Checklist:

  • I have followed the coding conventions.
  • I have updated or added tests to cover my changes.
  • I have updated the documentation if needed.
  • I have run pytest to check that all tests pass.
  • I have run pre-commit run --all-files to lint and format the code.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This is a cleanup PR that modernizes the development environment by replacing pip-based dependency management with micromamba/conda workflows. The changes focus on updating GitHub Actions workflows and configuration files while also applying code formatting improvements.

  • Updated GitHub Actions workflows to use micromamba with environment.yml
  • Added conda environment configuration file
  • Applied code formatting with Black (quote standardization and line wrapping)

Reviewed Changes

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

Show a summary per file
File Description
seagliderOG1/writers.py Code formatting: changed quotes and wrapped long log message
seagliderOG1/tools.py Code formatting: standardized quotes, wrapped long lines, removed blank lines
seagliderOG1/convertOG1.py Code formatting: standardized quotes, wrapped long lines, removed unused imports
requirements-dev.txt Restructured to reference base requirements.txt and simplified version constraints
environment.yml New conda environment configuration file
docs/source/conventions.md New comprehensive coding conventions documentation
README.md Enhanced with installation instructions, usage examples, and project structure
.github/workflows/tests.yml Updated to use micromamba with environment.yml instead of pip
.github/workflows/docs_deploy.yml Updated workflow to use micromamba environment
.github/workflows/docs.yml Updated workflow to use micromamba environment
.github/PULL_REQUEST_TEMPLATE.md Fixed broken link to coding conventions
.claude/settings.local.json New Claude AI configuration file

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread seagliderOG1/convertOG1.py
Comment thread README.md Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@eleanorfrajka eleanorfrajka merged commit cd72a43 into main Sep 20, 2025
7 checks passed
@eleanorfrajka eleanorfrajka deleted the eleanor-workflows-1 branch September 20, 2025 18:23
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