Skip to content

fix: set exitcode to 1 for analytics template validate/lint failures#248

Merged
sarabzadeh merged 7 commits intomainfrom
sarabzadeh-templatelintupdate-exitcode1
Mar 23, 2023
Merged

fix: set exitcode to 1 for analytics template validate/lint failures#248
sarabzadeh merged 7 commits intomainfrom
sarabzadeh-templatelintupdate-exitcode1

Conversation

@sarabzadeh
Copy link
Copy Markdown
Contributor

What does this PR do?

  • update template:lint and template:validate to exit non-0 if there are issues

What issues does this PR fix or reference?

@W-12705218@

@sarabzadeh sarabzadeh requested a review from a team as a code owner March 22, 2023 18:59
@sarabzadeh sarabzadeh self-assigned this Mar 22, 2023
Comment thread src/commands/analytics/template/lint.ts
Comment thread src/commands/analytics/template/lint.ts
Comment thread test/commands/template/lint.test.ts Outdated
Comment thread src/commands/analytics/template/lint.ts Outdated
Comment thread src/commands/analytics/template/validate.ts Outdated
@sarabzadeh sarabzadeh merged commit c43f666 into main Mar 23, 2023
@sarabzadeh sarabzadeh deleted the sarabzadeh-templatelintupdate-exitcode1 branch March 23, 2023 21:46
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.

5 participants