Skip to content

Only log 'forward' and 'backward' for 'change/0'#690

Merged
josevalim merged 1 commit into
elixir-ecto:masterfrom
nathanl:do_not_log_direction_for_up_and_down
Sep 15, 2025
Merged

Only log 'forward' and 'backward' for 'change/0'#690
josevalim merged 1 commit into
elixir-ecto:masterfrom
nathanl:do_not_log_direction_for_up_and_down

Conversation

@nathanl

@nathanl nathanl commented Sep 15, 2025

Copy link
Copy Markdown
Contributor

Existing logs like "running down/0 forward" look confusing, IMO. Only change/0 can be run backward or forward.

@josevalim josevalim merged commit 84dd74c into elixir-ecto:master Sep 15, 2025
8 of 11 checks passed
@josevalim

Copy link
Copy Markdown
Member

💚 💙 💜 💛 ❤️

@nathanl nathanl deleted the do_not_log_direction_for_up_and_down branch September 15, 2025 14:38
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.

2 participants