Matt Ulmer

I too see a similar issue using `citext`. ```bash /lib/python3.9/site-packages/eralchemy/sqla.py:84: SAWarning: Did not recognize type 'citext' of column 'name' Base.prepare(engine, reflect=True, name_for_scalar_relationship=name_for_scalar_relationship) ``` Is there a way around this? ERAlchemy:...

Yeah when I tried that earlier it failed as well with this error message: ``` Error: Error, failed to insert user test_user into instance test_instance: googleapi: Error 400: Invalid request:...

I have also run into this issue using [alembic](https://github.com/alembic/alembic). My use case relates to upgrading/downgrading my database schemas and I expected the `pre-rollback` hook to use the current image rather...

Has this been resolved at all? I'm having the same issue

@mreiferson thanks for the initial review! Apologies it took so long for me to circle back around to this, but it should be good for another look 👍

Hey @jehiah , > Can popTransaction take a 3rd error argument so we don't use magic values for some errors? I updated popTransaction to support this so now when there...

I did not, but I was able to confirm that [hasRepoCfg](https://github.com/runatlantis/atlantis/blob/a542aa8f2015e67957c5fdb6ec994080561aa1ec/server/events/project_command_builder.go#L346C3-L346C3) is always false for fork PRs. Since that is the case a clone will always performed, which seems to...

> > I did not, but I was able to confirm that [hasRepoCfg](https://github.com/runatlantis/atlantis/blob/a542aa8f2015e67957c5fdb6ec994080561aa1ec/server/events/project_command_builder.go#L346C3-L346C3) is always false for fork PRs. Since that is the case a clone will always performed, which...

Hi @jamengual, just bumping this thread in hopes this bug fix can get merged in!

@GenPage just letting you know that I fixed the merge conflicts!