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

Allow additional externals to be included #43

Merged

Conversation

jprosevear
Copy link
Contributor

@jprosevear jprosevear commented Feb 16, 2024

Fix for #42

@jprosevear jprosevear force-pushed the feature/add-optional-externals branch 2 times, most recently from e179bea to c1c56b1 Compare February 23, 2024 17:26
Copy link
Member

@crutchcorn crutchcorn left a comment

Choose a reason for hiding this comment

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

Approved! LGTM, thanks!

Copy link

nx-cloud bot commented Feb 25, 2024

☁️ Nx Cloud Report

CI 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 target

Sent with 💌 from NxCloud.

@crutchcorn
Copy link
Member

Err... We have some problems here with our Prettier config. Tagging in the big guns (@lachlancollins) to review

Lachlan, I'm running into issues where prettier is formatting every file despite looking good in CI/CD otherwise - on my Windows Machine

@crutchcorn crutchcorn requested review from lachlancollins and crutchcorn and removed request for crutchcorn February 25, 2024 06:22
@jprosevear
Copy link
Contributor Author

I can see how the workflow looks now, I'm looking into it.

@jprosevear jprosevear force-pushed the feature/add-optional-externals branch from c1c56b1 to 2bc3e79 Compare February 25, 2024 13:12
@jprosevear
Copy link
Contributor Author

Fixed formatting, looking at the build issue

@jprosevear jprosevear force-pushed the feature/add-optional-externals branch from 070525b to 589b0f5 Compare February 25, 2024 13:25
@jprosevear
Copy link
Contributor Author

@crutchcorn @lachlancollins resolved, pnpm test:pr now passes locally

Copy link
Member

@lachlancollins lachlancollins left a comment

Choose a reason for hiding this comment

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

LGTM!

@lachlancollins lachlancollins merged commit 34a3826 into TanStack:main Feb 26, 2024
2 checks passed
@jprosevear jprosevear deleted the feature/add-optional-externals branch June 21, 2024 13:40
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