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

add util for handling remix headers generally #810

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

nichtsam
Copy link
Contributor

@nichtsam nichtsam commented Aug 3, 2024

It's a bit tedious to manually merge Remix headers since the process is mostly the same.
This PR adds a utility to streamline that process.

Related Discussion #809
More details on the utility: https://gist.github.com/nichtsam/bff3a0e58f7edfad3985e9322582f67f

Test Plan

  • There are test cases for various functionalities.
  • Test cases with real use cases ensure the utility remains practical.

Checklist

  • Tests updated
  • Docs updated

@nichtsam nichtsam marked this pull request as draft August 3, 2024 14:41
@nichtsam
Copy link
Contributor Author

nichtsam commented Aug 3, 2024

Just for the record, reason for switching to draft: privatenumber/tsx#627
This is affecting vite and causing imports behavior to differentiate between tsx and node.

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.

1 participant