Ignore no git user.email if we're skipping git

[reavessm]

Advanced git-configs like includeIf break (see #238), this at least allows users to handle git on their own, later

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.

Problem/Feature

Please include a description of the problem or feature this PR is addressing.

Description of Changes:

• Ignore git user.email check if git config == skip

Checklist

• I have self-reviewed the changes being requested
• I have updated the documentation (check issue ticket 📋Update documentation if the PR introduces a new feature!📋 #218)

[ccoVeille]

This would be more readable and simple I think

[ccoVeille]

Suggested change

	if p.GitOptions.String() != flags.Skip {
	if !emailSet {
	fmt.Println("user.email is not set in git config.")
	fmt.Println("Please set up git config before trying again.")
	panic("\nGIT CONFIG ISSUE: user.email is not set in git config.\n")
	}
	}
	if !emailSet && p.GitOptions.String() != flags.Skip {
	fmt.Println("user.email is not set in git config.")
	fmt.Println("Please set up git config before trying again.")
	panic("\nGIT CONFIG ISSUE: user.email is not set in git config.\n")
	}

[Ujstor]

Good catch, this slipped under the radar. Tnx