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

annotate gemが正常に動かないことがある #2

Open
m11o opened this issue Nov 30, 2023 · 2 comments
Open

annotate gemが正常に動かないことがある #2

m11o opened this issue Nov 30, 2023 · 2 comments

Comments

@m11o
Copy link
Owner

m11o commented Nov 30, 2023

non-nullに変更した場合に、annotate gem が動かないことがある

可能性は2つ

  • db commentが付いているので、それが付いていると動かない (可能性大)
  • non-nullだと動かない

db commentはこういうやつ↓

#  ec_identifier(the user's id on the ec site) :string(255)
@m11o
Copy link
Owner Author

m11o commented Dec 3, 2023

ctran/annotate_models#1000 でPRを上げた

@m11o
Copy link
Owner Author

m11o commented Dec 3, 2023

原因はコメントがカラムについている場合に、起こることが分かった。
発生方法

  1. コメント付きのカラムを持つテーブルにAnnotationを追加
  2. コメント付きカラムを変更(non-null, default)
  3. Annotationが変更されない

理由は、新しいannotationと古いannotationで差分があるかどうかを確認する場合に、カラムごとの変更差分を正規表現で行っていたが、コメント付きカラムが実装された時に、この正規表現の () を追加しておらず、今回のバグが発生したと考えられる

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

No branches or pull requests

1 participant