Skip to content

Pre-release security audit#9

Merged
mrixlam merged 2 commits into
mainfrom
security-hardening
Jun 28, 2026
Merged

Pre-release security audit#9
mrixlam merged 2 commits into
mainfrom
security-hardening

Conversation

@mrixlam

@mrixlam mrixlam commented Jun 28, 2026

Copy link
Copy Markdown
Owner

Summary

This PR includes security audit results conducted prior to first official release. Also include tests for future reference.

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • [ x] New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that changes existing behavior)
  • Documentation update
  • Maintenance / refactor / CI

How has this been tested?

Ran repository's pytest with default lint

Checklist

  • [ x] My code follows the project style (passes ruff check and black --check).
  • [ x] Type checks pass (mypy mpasdiag/).
  • [ x] I have added or updated tests that cover my changes.
  • [ x] All tests pass locally (pytest tests/).
  • [ x] I have updated the documentation (README / docstrings) as needed.
  • I have added an entry to CHANGELOG.md under "Unreleased".
  • [ x] I have read the CONTRIBUTING guidelines.

@codecov

codecov Bot commented Jun 28, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 94.59459% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
mpasdiag/processing/remapping.py 84.61% 4 Missing ⚠️

📢 Thoughts on this report? Let us know!

@mrixlam mrixlam merged commit 268daaa into main Jun 28, 2026
11 checks passed
@sonarqubecloud

Copy link
Copy Markdown

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