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

check environment exists #45

Merged
merged 1 commit into from
Dec 14, 2021

Conversation

konifar
Copy link
Contributor

@konifar konifar commented Dec 6, 2021

Overview

environment is not required parameter but an error occurred if it is not set.

if (options.environment.match(idRegex)) {

Error: Cannot read property 'match' of null

Solution

I added environment check logic.

@maxx1e
Copy link

maxx1e commented Dec 13, 2021

Still have this issue. The only way to bypass is to add environment yaml option.

@matt-ball
Copy link
Owner

Thanks for this @konifar

@maxx1e this should help you 😄

@matt-ball matt-ball merged commit d2b5df6 into matt-ball:master Dec 14, 2021
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.

3 participants