Skip to content

Warning implementation for reviewing ops#777

Merged
zachdaniel merged 1 commit into
ash-project:mainfrom
colenelson0:feature/warnings-on-migration-ops
Jun 10, 2026
Merged

Warning implementation for reviewing ops#777
zachdaniel merged 1 commit into
ash-project:mainfrom
colenelson0:feature/warnings-on-migration-ops

Conversation

@colenelson0

@colenelson0 colenelson0 commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Contributor checklist

Leave anything that you believe does not apply unchecked.

  • I accept the AI Policy, or AI was not used in the creation of this PR.
  • Bug fixes include regression tests
  • Chores
  • Documentation changes
  • Features include unit/acceptance tests
  • Refactoring
  • Update dependencies

Notes about regression tests:

  • One unit test fails as expected by pre-existing documentation. The failure would occur before I made any changes to the repository and I was able to confirm that my implementation was unrelated to the test failure. The eight tests I added for my code additions all pass.
  • There were failures for reuse that were caused by untracked files/artifacts that surfaced during the process of running tests. Zach confirmed with me directly that I do not need to worry about this and that it will be handled later.

@zachdaniel zachdaniel merged commit 2b84c4c into ash-project:main Jun 10, 2026
99 of 109 checks passed
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