Skip to content

fix: implemented well-notes tests#246

Merged
jirhiker merged 2 commits into
stagingfrom
BDMS-233-jir
Nov 17, 2025
Merged

fix: implemented well-notes tests#246
jirhiker merged 2 commits into
stagingfrom
BDMS-233-jir

Conversation

@jirhiker

@jirhiker jirhiker commented Nov 14, 2025

Copy link
Copy Markdown
Member

Why

This PR addresses the following problem / context:

  • Use bullet points here

How

Implementation summary - the following was changed / added / removed:

  • Use bullet points here

Notes

Any special considerations, workarounds, or follow-up work to note?

  • Breaking database changes. DO NOT MERGE until ready to rerun transfer!

@codecov-commenter

codecov-commenter commented Nov 14, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Files with missing lines Coverage Δ
db/thing.py 100.00% <100.00%> (ø)
schemas/thing.py 97.72% <100.00%> (+0.05%) ⬆️

... and 19 files with indirect coverage changes

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread tests/features/steps/well-notes.py
Comment thread tests/features/steps/well-notes.py
Comment thread tests/features/steps/well-notes.py

@TylerAdamMartinez TylerAdamMartinez 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.

I'm unsure what the GPT AI code reviewer is getting at here, as I can see that you have defined the new fields in both the model and the response. All tests passed and overall lgtm.

@jirhiker jirhiker merged commit b29543a into staging Nov 17, 2025
6 checks passed
@TylerAdamMartinez TylerAdamMartinez deleted the BDMS-233-jir branch November 19, 2025 16:26
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