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.

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

Improve migration diff check and fix old migrations #5204

Draft
wants to merge 88 commits into
base: main
Choose a base branch
from
Draft

Improve migration diff check and fix old migrations #5204

wants to merge 88 commits into from

Conversation

dullbananas
Copy link
Collaborator

@dullbananas dullbananas commented Nov 17, 2024 •
edited
Loading

Previously part of #4673. That other PR must come before this one. I might remove the changes to old views.

@dullbananas
Update schema.rs
ca271ea
@dullbananas
Merge branch 'LemmyNet:main' into migration-runner
2775242
@dullbananas
rename
f2a6d73
@dullbananas
stuff
691bce0
@dullbananas
Merge remote-tracking branch 'upstream/main' into migration-runner
1aab92c
@dullbananas
finish new implementation of schema_setup::run (not including revert,…
e8e354c 
… test, etc.)
@dullbananas
fmt
402ab14
@dullbananas
refactor
08ae13b
@dullbananas
fix sql
542f81d
@dullbananas
migriation run command
d0d8139
@dullbananas
use trigger on migrations table
22ac8c5
@dullbananas
add Options with disable_migrations field for test
27b289c
@dullbananas
rename to enable_forbid_diesel_cli_trigger
614e0e5
@dullbananas
fix
5d8af0d
@dullbananas
Merge remote-tracking branch 'upstream/main' into migration-runner
1af517f
@dullbananas
fix merge
16dcbc7
@dullbananas
diff_checker (partial)
6709882
@dullbananas
Revert "diff_checker (partial)"
d4bdda5 
This reverts commit 6709882.
@dullbananas
Revert "Revert "diff_checker (partial)""
8e0bbd6 
This reverts commit d4bdda5.
@dullbananas
diff check
1692fd6
@dullbananas
improve schema diff
84bf57e
@dullbananas
timestamp replacement
d18c2c9
@dullbananas
ignore column order
314e744
@dullbananas
remove fedi_name default
80ca94e
@dullbananas
stuff
06b01ff
@dullbananas
improve diff
7977ee1
@dullbananas
stuff
ced9bb5
@dullbananas
attempt parallel pg_dump
592a127
@dullbananas
attempt 2
a909d2d
@dullbananas
Revert "attempt 2"
47a4c35 
This reverts commit a909d2d.
@dullbananas
Update .woodpecker.yml
1e56bde
@dullbananas
Update .woodpecker.yml
1a71897
@dullbananas
try to fix env var
d57d293
@dullbananas
Update diff_check.rs
56e22b0
@dullbananas
Merge branch 'main' into migration-runner
db856ef
@dullbananas
Remove condition that's not needed anymore
d82bfcc
@dullbananas
clippy
c694da9
@dullbananas
exclude views and fast tables
120be51
@dullbananas
revert some migration changes
317097c
@dullbananas
fix
b6102dc
@dullbananas
fmt
1a18350
@dullbananas
re-attempt checking character after skipped trigger name, and make co…
d592e0f 
…de less confusing
@dullbananas
fmt
3daa0d6
@dullbananas
Merge branch 'main' into migration-runner
0b8790c
@dullbananas
fix
ce1cf3e
@dullbananas
rerun ci
45562bf
@dullbananas
rerun ci
b176340
@dullbananas
fix strip_prefix order
a242333
@dullbananas
Merge remote-tracking branch 'upstream/main' into migration-runner
7734ba4
@dullbananas
Merge remote-tracking branch 'upstream/main' into migration-runner
de01635
@dullbananas
fix weird big Cargo.lock change by running `git checkout upstream/mai…
de0e025 
…n Cargo.lock` then letting it auto update again
@dullbananas
Merge remote-tracking branch 'upstream/main' into migration-runner
4428c61
@dullbananas
fix
84b161d
@dullbananas
Merge remote-tracking branch 'upstream/main' into migration-runner
ecb700e
@dullbananas
remove installation commands that were removed in main branch
fd65234
@dullbananas
Revert "remove installation commands that were removed in main branch"
6345b2d 
This reverts commit fd65234.
@dullbananas
Merge upstream/main into migration-runner (using imerge)
f636f1a
@dullbananas
Merge remote-tracking branch 'upstream/main' into migration-runner
674a12e
@dullbananas
move create_database_user woodpecker step to make diff less weird
ffe21da
@dullbananas dullbananas mentioned this pull request Nov 17, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dessalines dessalines Awaiting requested review from dessalines dessalines will be requested when the pull request is marked ready for review dessalines is a code owner

@phiresky phiresky Awaiting requested review from phiresky phiresky will be requested when the pull request is marked ready for review phiresky is a code owner

@Nutomic Nutomic Awaiting requested review from Nutomic Nutomic will be requested when the pull request is marked ready for review Nutomic is a code owner

@SleeplessOne1917 SleeplessOne1917 Awaiting requested review from SleeplessOne1917 SleeplessOne1917 will be requested when the pull request is marked ready for review SleeplessOne1917 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dullbananas