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

chore: copy some aspect files to the client side to prepare for templating #6842

Merged
merged 20 commits into from
Oct 9, 2024

Conversation

xuansontrinh
Copy link
Collaborator

Description of this change

The purpose of this PR is to ready the plugin for templating some aspect files, e.g. java_info.bzl.
This is just an intermediate step and the next change may considerably change what is put here.

@github-actions github-actions bot added product: CLion CLion plugin product: IntelliJ IntelliJ plugin product: GoLand GoLand plugin awaiting-review Awaiting review from Bazel team on PRs labels Oct 4, 2024
@xuansontrinh xuansontrinh force-pushed the son/copy_java_info_to_project_side branch from a47f340 to 4084549 Compare October 4, 2024 07:42
@xuansontrinh xuansontrinh force-pushed the son/copy_java_info_to_project_side branch from 4084549 to 0907230 Compare October 4, 2024 07:50
@xuansontrinh xuansontrinh marked this pull request as draft October 4, 2024 08:52
@xuansontrinh xuansontrinh force-pushed the son/copy_java_info_to_project_side branch from ea0279c to 10b801d Compare October 4, 2024 17:22
@xuansontrinh xuansontrinh force-pushed the son/copy_java_info_to_project_side branch from ba77815 to 4d012c9 Compare October 5, 2024 19:22
@xuansontrinh xuansontrinh force-pushed the son/copy_java_info_to_project_side branch from 4d012c9 to 8d6b4c4 Compare October 5, 2024 19:24
@tpasternak
Copy link
Collaborator

The test is most likely false positive. Let's disable it for now, so @xuansontrinh can move forward and I (or someone else) will take care of the test

@xuansontrinh xuansontrinh marked this pull request as ready for review October 9, 2024 11:12
@tpasternak tpasternak merged commit 893eaa8 into master Oct 9, 2024
8 checks passed
@github-actions github-actions bot removed the awaiting-review Awaiting review from Bazel team on PRs label Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
product: CLion CLion plugin product: GoLand GoLand plugin product: IntelliJ IntelliJ plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants