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

Feature/nest validate pipeline supports directories and node versions #15

Conversation

skifahrer
Copy link
Contributor

validation pipeline now has more options - choosing working directory, node version and warning levels.

@skifahrer skifahrer requested a review from vidriduch May 22, 2024 14:53
Copy link
Contributor

@vidriduch vidriduch left a comment

Choose a reason for hiding this comment

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

I'm fine with this. Besides one question/comment LGTM

level: warning
workdir: ./src
level: ${{inputs.level}}
workdir: ${{ inputs.directory }}/src
Copy link
Contributor

Choose a reason for hiding this comment

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

Are we absolutely sure that all of our Nest projects have /src ? Isn't there a way how we can get/set this directory automatically rather then hardcoding it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

true, moved to default value

@skifahrer skifahrer merged commit 7bcf000 into master May 27, 2024
@skifahrer skifahrer deleted the feature/nest-validate-pipeline-supports-directories-and-node-versions branch May 27, 2024 09:00
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