Skip to content

fix(iac) : infra to support bun check#574

Merged
whutchinson98 merged 4 commits intomainfrom
hutch/bac-162-fix-infra-to-support-bun-check
Dec 11, 2025
Merged

fix(iac) : infra to support bun check#574
whutchinson98 merged 4 commits intomainfrom
hutch/bac-162-fix-infra-to-support-bun-check

Conversation

@whutchinson98
Copy link
Copy Markdown
Member

Summary

This PR updates all of our paths to go back to stupid relative paths since it is the only way to get typechecking to work.

This PR also removes an unused stack networking and moves the fusionauth stack into the stacks folder.

Note: this PR does not touch web-app stack as it's not part of the turbo workspace at the moment. That will come in a follow-up PR

Screenshots, GIFs, and Videos

@whutchinson98 whutchinson98 requested a review from a team as a code owner December 11, 2025 20:45
@linear
Copy link
Copy Markdown

linear bot commented Dec 11, 2025

Copy link
Copy Markdown
Contributor

@gbirman gbirman left a comment

Choose a reason for hiding this comment

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

oof, how do you expect to enforce this property with future code?

@whutchinson98
Copy link
Copy Markdown
Member Author

whutchinson98 commented Dec 11, 2025

oof, how do you expect to enforce this property with future code?

code reviews. also all the relative @ stuff is completely removed so it won't work

@whutchinson98 whutchinson98 merged commit 5b1d4cd into main Dec 11, 2025
19 checks passed
@whutchinson98 whutchinson98 deleted the hutch/bac-162-fix-infra-to-support-bun-check branch December 11, 2025 20:51
@seanaye
Copy link
Copy Markdown
Contributor

seanaye commented Dec 12, 2025

oof, how do you expect to enforce this property with future code?

The relative path is used to get the compiler to pass so the compiler will enforce this. The goal isn't to enforce relative paths, the goal is to typecheck all the code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants