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

fix: include code.js in tsconfig #623

Merged
merged 1 commit into from
Dec 4, 2024
Merged

Conversation

ecshreve
Copy link
Contributor

@ecshreve ecshreve commented Dec 4, 2024

What does this do?

  • include the /src/js directory so it gets picked up at buildtime
  • use suggested inline ignore comment
  • also update copyAssets to copy openapi spec to correct directory

How was it tested?

localhost
landing

  • also built the docker image and ran it locally to double check

Is there a Github issue this is resolving?

Was any impacted documentation updated to reflect this change?

n/a

Here's a fun image for your troubles

random photo - update me

- use suggested inline ignore comment
- update copyAssets to copy openapi spec to
  correct directory
@bagelbits bagelbits merged commit 3a03353 into 5e-bits:main Dec 4, 2024
9 checks passed
Copy link

github-actions bot commented Dec 4, 2024

🎉 This PR is included in version 3.10.4 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

2 participants