Skip to content

Conversation

@breardon2011
Copy link
Contributor

@breardon2011 breardon2011 commented Oct 22, 2025

migrator, package atlas in dockerfile,

set CGO in dockerfile, was 0, but need 1 for sqlite
fix in memory system id

tested in dockerfile

🧠 AI Assistance Disclosure Policy

Important

Inspired by ghostty.
If you used any AI assistance while contributing to Digger, you must disclose it in this PR.


✅ AI Disclosure Checklist

  • I understand that all AI assistance must be disclosed.
  • I did not use AI tools in this contribution.
  • I used AI tools and have disclosed details below.

Details (if applicable):
I used cursor in a directed way

"do x, y, z"
"is this adherent to clean architecture patterning"
etc


💡 Notes

  • Trivial auto-completions (single words, short phrases) don’t need disclosure.
  • Contributors must understand and take responsibility for any AI-assisted code.
  • Failure to disclose is considered disrespectful to maintainers and may delay review.

@breardon2011 breardon2011 marked this pull request as ready for review October 23, 2025 00:14
Copy link
Contributor

@motatoes motatoes left a comment

Choose a reason for hiding this comment

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

i don't think we should have the automigrator fallback, it makes things much less predictable with DB migrations. I would say lets only rely on atlas

@breardon2011
Copy link
Contributor Author

yeah that makes sense, i'll take it out

@breardon2011 breardon2011 requested a review from motatoes October 23, 2025 00:47
@breardon2011 breardon2011 changed the title Atlas with AutoMigrate fallback Atlas with no fallback Oct 23, 2025
@breardon2011 breardon2011 changed the title Atlas with no fallback Atlas Oct 23, 2025
@breardon2011
Copy link
Contributor Author

added the migrations and workflows to validate.

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