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

DAT 3464 #955

Merged
merged 8 commits into from Jan 2, 2020
Merged

DAT 3464 #955

merged 8 commits into from Jan 2, 2020

Conversation

wwillard7800
Copy link
Contributor

No description provided.

@madorb
Copy link

madorb commented Dec 27, 2019

I'm curious if it's intentional that there's no description provided with these DAT merges? I assume al the details are stored in an internal issue tracker.

It seems like it would be useful to provide a publicly facing description for each of these merge requests so that potential community committers have a better understanding of why particular code changes were made?

@nvoxland
Copy link
Contributor

Yes, sorry. We are making some internal process changes lately, and changes to the pull request descriptions and issue tracking is things we are still working on. New pull requests we make going forward will be more descriptive and useful.

@madorb
Copy link

madorb commented Jan 2, 2020

Thanks @nvoxland! Appreciate all the work you & the Datical team do for the liquibase community!

# Conflicts:
#	liquibase-core/src/main/java/liquibase/integration/commandline/Main.java
#	liquibase-core/src/test/java/liquibase/integration/commandline/MainTest.java
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.

None yet

4 participants