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

[pull] main from rust-db:main #7

Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open

Conversation

pull[bot]
Copy link

@pull pull bot commented Mar 1, 2024

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

* apply clippy lints

* update CI to address MSRV
@pull pull bot added the ⤵️ pull label Mar 1, 2024
rjkz808 and others added 7 commits March 2, 2024 21:12
* Fixes a bug in `get_last_applied_migration` due to an incorrect SQL
  query that partially uses dynamic table names and partially hardcodes
  the default one.

* Adds a new utility function that enables dynamic migration discovery
  where embedding is not desirable.
Make the relative link to example work from the refinery folder
* generate EmbeddedMigrations enum

* create enum from migration

* consolidate regex and remove lazy_static

* fmt, cleanup

* remove unused trait

* use From for migration enum

* remove unneeded usings

* add feature flag, update example project

* fmt

* Update refinery_core/src/util.rs

Co-authored-by: João Oliveira <[email protected]>

* Update refinery_core/src/util.rs

Co-authored-by: João Oliveira <[email protected]>

* Update refinery_core/src/util.rs

Co-authored-by: João Oliveira <[email protected]>

* Update refinery_core/src/util.rs

Co-authored-by: João Oliveira <[email protected]>

---------

Co-authored-by: João Oliveira <[email protected]>
jxs and others added 11 commits April 3, 2024 19:59
* Allow Postgres tests to be run on a different database

Not everyone has a "scratch" PostgreSQL running on localhost:5432 for refinery to scribble all over.
Now you can specify an arbitrary PostgreSQL server to work on with the `DB_URI` environment variable (which appears to be what `refinery-cli` already uses) to test in.

* Improve DB reset process

* Use a more appropriate name for the function that does the work
* Clean just the `public` schema, rather than drop/create the whole DB.  This means that running the tests no longer requires superuser privs, and that we don't have to temporarily hide out in `template1`.
* Drop the `catch_unwind`, because it's needed any more.

---------

Co-authored-by: João Oliveira <[email protected]>
* Enable math functions for SQLite

* Update .cargo/config.toml

---------

Co-authored-by: Pamplemousse <[email protected]>
Co-authored-by: João Oliveira <[email protected]>
remove no longer required async section and mention external drivers
This allows avoiding double iteration when calling migrate with the grouped option active
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.