Skip to content

Add an optional argument in the generate-model-yml macro to preserve data type case of the target database #253

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
fionayz wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Add an optional argument in the generate-model-yml macro to preserve data type case of the target database #253

fionayz wants to merge 5 commits into from

Conversation

@fionayz
Copy link

@fionayz fionayz commented Apr 10, 2025

Problem

resolves #252

Solution

Checklist

  • This code is associated with an issue which has been triaged and accepted for development.
  • I have read the contributing guide and understand what's expected of me
  • I have run this code in development and it appears to resolve the stated issue
  • This PR includes tests, or tests are not required/relevant for this PR
  • I have updated the README.md (if applicable)

@fionayz fionayz requested a review from a team as a code owner April 10, 2025 16:31
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Apr 10, 2025
@fionayz fionayz changed the title Fionayz/resolve 252 Add an optional argument in the generate-model-yml macro to preserve data type case of the target database Apr 10, 2025
@fionayz fionayz marked this pull request as draft April 10, 2025 17:08
@fionayz
Copy link
Author

fionayz commented Apr 11, 2025 •
edited
Loading

The failed tests don't seem to be related to changes in this PR. I'm a bit confused at the moment, likely need extra eyes

@fionayz fionayz marked this pull request as ready for review April 11, 2025 01:58
@github-actions
Copy link

github-actions bot commented Oct 8, 2025

This PR has been marked as Stale because it has been open with no activity as of late. If you would like the PR to remain open, please comment on the PR or else it will be closed in 7 days.

@github-actions github-actions bot added the Stale label Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

size:L This PR changes 100-499 lines, ignoring generated files. Stale

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add an option in the generate-model-yml macro to preserve data type case of the target database.

1 participant

@fionayz