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

cleanup #44

Merged
merged 3 commits into from
Nov 16, 2017
Merged

cleanup #44

merged 3 commits into from
Nov 16, 2017

Conversation

sherl0cks
Copy link
Contributor

@sherl0cks sherl0cks commented Nov 14, 2017

resolves #3
resolves #4
resolves #13
resolves #37

Ready to go

@sherl0cks sherl0cks changed the title WIP - cleanup cleanup Nov 14, 2017
@sherl0cks
Copy link
Contributor Author

yikes - we have some regressions. hold on.

@sherl0cks sherl0cks changed the title cleanup wip - cleanup Nov 14, 2017
}
},
{
"apiVersion": "v1",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

FYI this probably should have happened with #42

@sherl0cks sherl0cks changed the title wip - cleanup cleanup Nov 14, 2017
@sherl0cks
Copy link
Contributor Author

Ok looks good now

@oybed
Copy link
Contributor

oybed commented Nov 16, 2017

@sherl0cks a lot of great stuff and I'm good with it. However, to aid in case someone is still relying on the "old stuff", I cut a pre-release tag v0.1.0 that we can point people to in case they need anything from the older version(s). At this point I'm good to merge this as-is. I have one minor suggestion, but I'll submit a follow-up PR to take care of that one.

https://github.com/rht-labs/labs-ci-cd/releases/tag/v0.1.0

@oybed oybed merged commit 78c56d0 into rht-labs:master Nov 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants