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

chore: simplify supported projects validation on test #1292

Merged
merged 1 commit into from
Jul 28, 2020

Conversation

lili2311
Copy link
Contributor

@lili2311 lili2311 commented Jul 28, 2020

  • Ready for review
  • Follows CONTRIBUTING rules
  • Reviewed by Snyk internal team

What does this PR do?

Simplify the validation logic & extract it for readability.

Blocked by #1292

@lili2311 lili2311 requested review from a team as code owners July 28, 2020 09:36
@lili2311 lili2311 self-assigned this Jul 28, 2020
@ghost ghost requested review from anthogez and ekbsnyk July 28, 2020 09:37
@lili2311 lili2311 added the 🚧 WIP Work In Progress label Jul 28, 2020
@lili2311 lili2311 force-pushed the feat/refactor-project-type-validation branch 2 times, most recently from 5e763a6 to 935b7fc Compare July 28, 2020 11:30
@lili2311 lili2311 changed the title feat: refactor projectType validation chore: simplify supported projects validation on test Jul 28, 2020
@snyksec
Copy link

snyksec commented Jul 28, 2020

Messages
📖

This PR will not trigger a new version. It doesn't include any commit message with feat or fix.

Generated by 🚫 dangerJS against a794bde

@lili2311 lili2311 force-pushed the feat/refactor-project-type-validation branch from 935b7fc to 9452f0d Compare July 28, 2020 12:04
@lili2311 lili2311 added ☠️ tech services and removed 🚧 WIP Work In Progress labels Jul 28, 2020
@lili2311 lili2311 force-pushed the feat/refactor-project-type-validation branch from 9452f0d to a794bde Compare July 28, 2020 12:06
@lili2311 lili2311 merged commit 19e7c79 into master Jul 28, 2020
@lili2311 lili2311 deleted the feat/refactor-project-type-validation branch July 28, 2020 16:23
@snyksec
Copy link

snyksec commented Jul 28, 2020

🎉 This PR is included in version 1.369.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants