Skip to content
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.

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

#396 changes in gen.sh and models #401

Merged
merged 1 commit into from
Nov 2, 2023

Conversation

evassilyev
Copy link
Contributor

@evassilyev evassilyev commented Oct 29, 2023

Path to fix the issue #396.

@kenshaw please, take a look.
I hope it would be useful and I fixed all files needed.

  1. I made changes in gen.sh
  2. sh gen.sh models
  3. restored all the code for other DBs (I have Postgres only) that was erased by gen.sh
  4. build xo binary and checked that it generates code without issue

@kenshaw kenshaw merged commit 371c056 into xo:master Nov 2, 2023
1 check passed
@kenshaw
Copy link
Member

kenshaw commented Nov 2, 2023

@evassilyev this has been merged. Would it be possible to also do a PR separately with an example of where issue #396 pops up in one of the examples?

@evassilyev evassilyev deleted the #396-fix-same-name-fk-issue branch November 4, 2023 20:49
@evassilyev
Copy link
Contributor Author

evassilyev commented Nov 4, 2023

@kenshaw hi, separated PR with an example which reproduces the issue - 402.

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.

None yet

2 participants