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

installer automation? #5433

Merged
merged 7 commits into from
Apr 29, 2018
Merged

installer automation? #5433

merged 7 commits into from
Apr 29, 2018

Conversation

tiagom62
Copy link
Contributor

@tiagom62 tiagom62 commented Apr 26, 2018

Mostly formatting changes. Trying to align with https://google.github.io/styleguide/shell.xml
Support cli switches to hopefully speed up/automate testing in the future.
Made the verbose option infinity more useful using set -x

View here with whitespace changes ignored.. https://github.com/snipe/snipe-it/pull/5433/files?w=1

Now includes a Vagrantfile for easier testing!

@tiagom62 tiagom62 requested a review from snipe as a code owner April 26, 2018 02:28
@tiagom62 tiagom62 changed the title Mostly formatting ~Mostly formatting~ Apr 28, 2018
@tiagom62 tiagom62 changed the title ~Mostly formatting~ installer automation? Apr 28, 2018
@tiagom62 tiagom62 force-pushed the installer_formatting branch from 57996c0 to 5db40a7 Compare April 28, 2018 05:51
@tiagom62 tiagom62 force-pushed the installer_formatting branch from 5db40a7 to c1f6ca0 Compare April 28, 2018 05:52
@snipe snipe merged commit 6ad3d40 into snipe:develop Apr 29, 2018
@snipe
Copy link
Owner

snipe commented Apr 29, 2018

cumberbatch-thank-you

@tiagom62 tiagom62 deleted the installer_formatting branch April 29, 2018 15:59
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