-
-
Notifications
You must be signed in to change notification settings - Fork 13
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
Allow additional externals to be included #43
Allow additional externals to be included #43
Conversation
e179bea
to
c1c56b1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved! LGTM, thanks!
☁️ Nx Cloud ReportCI is running/has finished running commands for commit 589b0f5. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this CI Pipeline Execution ✅ Successfully ran 1 targetSent with 💌 from NxCloud. |
Err... We have some problems here with our Prettier config. Tagging in the big guns (@lachlancollins) to review
|
I can see how the workflow looks now, I'm looking into it. |
c1c56b1
to
2bc3e79
Compare
Fixed formatting, looking at the build issue |
070525b
to
589b0f5
Compare
@crutchcorn @lachlancollins resolved, pnpm test:pr now passes locally |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Fix for #42