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

Github Actions with minor fixes #11

Merged
merged 10 commits into from
Sep 23, 2019
Merged

Github Actions with minor fixes #11

merged 10 commits into from
Sep 23, 2019

Conversation

mcred
Copy link
Collaborator

@mcred mcred commented Sep 23, 2019

Issues Addressed

Fixes #6
Windows unit tests fail when testing for line breaks.
Run fails if the docker container is not already pulled.

Proposed Changes

Rather than use \n as a new line System.lineSeparator() is used.
Pulls container during run to make sure it's local before trying to inspect.

Testing

Migrate a container and inspect the volumes. The first one will start with v0 and increment up.
Run the unit test in windows to see the line break fixes.
run a new repo that you do not have locally.

Derek Smart and others added 9 commits September 18, 2019 10:58
Signed-off-by: Derek Smart <[email protected]>
Signed-off-by: Derek Smart <[email protected]>
Signed-off-by: Derek Smart <[email protected]>
Signed-off-by: Derek Smart <[email protected]>
Signed-off-by: Derek Smart <[email protected]>
Signed-off-by: Derek Smart <[email protected]>
Signed-off-by: Derek Smart <[email protected]>
@mcred mcred requested a review from a team as a code owner September 23, 2019 20:37
Copy link
Contributor

@eschrock eschrock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make sure to restore windows tests when adding windows support back in.

@eschrock eschrock merged commit 27891cd into titan-data:master Sep 23, 2019
mcred referenced this pull request in mcred/titan Sep 24, 2019
Github Actions with minor fixes (#11)
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.

migrate places data in incorrect volume
2 participants