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 Bazel ts support #8499

Merged
merged 1 commit into from
Jan 23, 2025
Merged

Conversation

mering
Copy link
Contributor

@mering mering commented Jan 22, 2025

Fixes the following error:

LOC/flatbuffers/tests/ts/typescript_keywords.fbs:2: 36: error: unable to load include file: reflection/reflection.fbs

@mering mering force-pushed the fix-bazel-reflection-path branch from 52027c5 to 91982d2 Compare January 23, 2025 10:37
@github-actions github-actions bot added the CI Continuous Integration label Jan 23, 2025
@mering mering force-pushed the fix-bazel-reflection-path branch 2 times, most recently from 50ef412 to 2c21a81 Compare January 23, 2025 12:02
@mering mering changed the title Fix reflection.fbs import path Fix ts support Jan 23, 2025
@mering mering force-pushed the fix-bazel-reflection-path branch 3 times, most recently from 277777b to 7f216cd Compare January 23, 2025 12:46
@mering mering mentioned this pull request Jan 23, 2025
@mering mering force-pushed the fix-bazel-reflection-path branch 7 times, most recently from 8a7ae42 to 8c4bfae Compare January 23, 2025 15:56
@mering mering marked this pull request as ready for review January 23, 2025 15:56
@mering mering changed the title Fix ts support Fix Bazel ts support Jan 23, 2025
@mering mering force-pushed the fix-bazel-reflection-path branch 2 times, most recently from 4a5f612 to a5f3974 Compare January 23, 2025 17:58
build_defs.bzl Show resolved Hide resolved
@mering mering force-pushed the fix-bazel-reflection-path branch from a5f3974 to b3ec2fd Compare January 23, 2025 19:33
@github-actions github-actions bot removed the CI Continuous Integration label Jan 23, 2025
We need to copy the .fbs files into the package used for .bfbs files.
This is necessary as flatc doesn't provide support to specify the full
output file name for an .fbs file in a different folder.
I tried OUTPUT_FILE env var but this doesn't seem to be honored by
flatc.
@mering mering force-pushed the fix-bazel-reflection-path branch from b3ec2fd to eb5cd8b Compare January 23, 2025 19:38
@dbaileychess dbaileychess enabled auto-merge (squash) January 23, 2025 19:41
@dbaileychess dbaileychess merged commit 33a15d6 into google:master Jan 23, 2025
49 checks passed
@mering mering deleted the fix-bazel-reflection-path branch January 23, 2025 21:01
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.

2 participants