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.

Sign up for GitHub

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

Jump to bottom

DRAFT Compile new files #79

Draft
wants to merge 17 commits into
base: wix-master
Choose a base branch
from
Draft

DRAFT Compile new files #79

wants to merge 17 commits into from

Conversation

mildagle
Copy link

No description provided.

@WixBuildServer WixBuildServer force-pushed the wix-master branch 3 times, most recently from 380a8e9 to 60b5109 Compare February 4, 2023 05:06
@WixBuildServer WixBuildServer force-pushed the wix-master branch 5 times, most recently from 150a6fd to f14b17f Compare February 14, 2023 05:06
@WixBuildServer WixBuildServer force-pushed the wix-master branch 3 times, most recently from 207d649 to ef463dd Compare February 17, 2023 05:06
@WixBuildServer WixBuildServer force-pushed the wix-master branch 4 times, most recently from 187874c to 269d8f2 Compare February 28, 2023 05:06
ZachiNachshon and others added 14 commits March 1, 2023 05:09
@ZachiNachshon @WixBuildServer
[src] Arbitrary deployable JAR task should not upload local results t…
050cfa9 
…o remote cache
@ittaiz @WixBuildServer
only filters labels according to parent packages for LanguageClass.C (#…
3b2640b 
…47)

* only filters labels according to parent packages for LanguageClass.C
This is to keep supporting CPP need while also supporting aggregate targets
More info:
https://bazelbuild.slack.com/archives/CM8JQCANN/p1566481600003600
Filter was deliberately added to fix an issue with C++ headers.
The original bug was that opening the BUILD file for foo/bar.h caused it to open baz/qux/BUILD because there’s a cc_library target in baz/qux that includes foo/bar.h in its headers. The source-to-target look up queries the index for the target during the last sync which compiled the source file. In this case, it was the cc_library target in baz/qux.
So the fix was to ensure that the BUILD file is always for the parent package (foo/bar)
@ittaiz @WixBuildServer
derive_targets_from_directories now defaults to false on project import
3d3150a
@liucijus @WixBuildServer
Scala source attachment pt2
e60198a 
Don't use ijars as they break source navigation

- ignore java_import /_ijar/ as its original location is not know
- replace -ijar with sibling full jar
- replace -hjar with sibling full jar

https://youtrack.jetbrains.com/issue/SCL-16975/Navigation-to-attached-Scala-sources-for-interface-jar-does-not-work
@liucijus @WixBuildServer
Scala source attachment pt3
68b1f05 
Add support for scala source attachment
@liucijus @WixBuildServer
Scala source attahcment pt4
0c25169 
Add source root detection for Java and Scala source jars. Detection is required when dire4ctory structure in source jar has deeper structure than source packages.

Source attachement changes for 2022.3 support
@aradchykov @WixBuildServer
Prelude support (#53)
0e21d7b
@povilas @WixBuildServer
Display notification when JDK is not configured.
0af2813
@povilas @WixBuildServer
Notify user when bazel binary cannot be executed
9fc269a
@liucijus @WixBuildServer
Support full range major version also for EAP
1b0ab92
@simuons @WixBuildServer
Expose ProtoIdeInfo for proto_library during sync
bda8654 
`idea.plugin.protoeditor` needs to be aware of source roots of proto
libraries

Add proto files to target sources
@liucijus @WixBuildServer
Add env var to pass additional debug flags for Java
42c52fa
@liucijus @WixBuildServer
Fix handling of Wix specific test configuration
df17c1c
@ittaiz @WixBuildServer
Fast test supports Scala junit
1b5492c 
Via running bloop and BSP

Fast test supports ITs
Location substitution supports runpath(s) in addition to location(s)
Still includes a hack of the env variable of BUILD_TOOL which we need to see how to remove but at least a smaller hack than before

Full Compile on proto file changes

don't update sources after proto change already exists
@WixBuildServer WixBuildServer force-pushed the wix-master branch 3 times, most recently from 0bc3c18 to daee0ed Compare January 29, 2025 05:02
@WixBuildServer WixBuildServer force-pushed the wix-master branch 2 times, most recently from 8d03cf9 to 63283c9 Compare February 2, 2025 05:02
@WixBuildServer WixBuildServer force-pushed the wix-master branch 3 times, most recently from 0bff421 to 506fda1 Compare February 8, 2025 05:02
@WixBuildServer WixBuildServer force-pushed the wix-master branch 6 times, most recently from f1cc266 to 384f6a0 Compare February 19, 2025 05:02
@simuons simuons force-pushed the wix-master branch 2 times, most recently from 7867317 to 20797f6 Compare March 10, 2025 08:45
@WixBuildServer WixBuildServer force-pushed the wix-master branch 3 times, most recently from 876fe80 to 3be7690 Compare March 15, 2025 05:01
@WixBuildServer WixBuildServer force-pushed the wix-master branch 5 times, most recently from 3be5785 to 2612174 Compare March 25, 2025 05:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@mildagle @ZachiNachshon @ittaiz @liucijus @aradchykov @povilas @simuons