Skip to content

Allow additional externals to be included#43

Merged
lachlancollins merged 1 commit intoTanStack:mainfrom
codalio:feature/add-optional-externals
Feb 26, 2024
Merged

Allow additional externals to be included#43
lachlancollins merged 1 commit intoTanStack:mainfrom
codalio:feature/add-optional-externals

Conversation

@jprosevear
Copy link
Copy Markdown
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
Copy Markdown
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!

@nx-cloud
Copy link
Copy Markdown

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
Copy Markdown
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 crutchcorn and lachlancollins and removed request for crutchcorn February 25, 2024 06:22
@jprosevear
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor Author

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

Copy link
Copy Markdown
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
@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