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

sandbox #1

Draft
wants to merge 58 commits into
base: master
Choose a base branch
from
Draft

sandbox #1

wants to merge 58 commits into from

Conversation

mbien
Copy link
Owner

@mbien mbien commented Nov 8, 2023

No description provided.

@mbien mbien added the LSP label Nov 8, 2023
@mbien mbien force-pushed the clone-build branch 2 times, most recently from e18b4cd to 637fd04 Compare November 8, 2023 10:19
@mbien mbien added Gradle and removed LSP labels Feb 2, 2024
Repository owner locked and limited conversation to collaborators Feb 2, 2024
Repository owner unlocked this conversation Feb 2, 2024
@mbien mbien force-pushed the clone-build branch 2 times, most recently from dec9015 to 66d417d Compare February 2, 2024 15:59
@mbien mbien added the LSP label Feb 2, 2024
@mbien mbien force-pushed the clone-build branch 3 times, most recently from e5282b5 to a127123 Compare February 10, 2024 04:01
@mbien mbien removed the Platform label Feb 10, 2024
@mbien mbien removed the ci:all-tests label Apr 1, 2024
jGauravGupta and others added 13 commits October 20, 2024 11:36
FISH-7079 Copy compiled resources from classes directory to exploded binary
PR review changes - inline return condition
Constructor Signature update
Remove log
Add generic type
Fix Single-file-java-launcher test and add module to CI
Update ASM from 9.7 to 9.7.1
…ation_file_owner

Making minor code modifications for better readability and validation
Support for Payara Server instance running on WSL
Removed changes in comments while renaming
Increment spec versions for Apache NetBeans 25 development
issue template: enable 24 rc selection
….source.base

Fixed tests failures in the modules java.hints and java.source.base on
Windows due to CRLF-LF mismatches between the expected and actual
outputs.

1. *VanillaCompileWorkerTest.java*:
    - transform `System.lineSeparator()` to LF on the actual outputs.
2. *TreeRuleTestBase.java*:
    - change the "ignore whitespace" transformation of the actual output
      to use `\\s+` instead of only tabs and LFs.
3. *.gitattributes*:
    - Mark all test golden files i.e. expected outputs, in the
      java.hints module to have only LF line endings.
    - This aids tests extending *TreeRuleTestBase*
    - Thanks to @mbien for guidance on this.

Signed-off-by: Siddharth Srinivasan <[email protected]>
lahodaj and others added 6 commits October 29, 2024 08:11
…code_bug

Fix code-folding for LSP clients that support line-folding only
…_windows

Fix tests failure due to CRLF mismatch on Windows in java.hints, java.source.base
The javac outputs of the following tests in java/java.hints
ErrorHintsProviderTest were updated for non-mac OSes due to changes in
JDK 7 updates:
- testShortErrors5: [jdk-6968793](openjdk/jdk@b77effad6c6)
- testTestShortErrorsSVUIDWarning: [jdk-6957438](openjdk/jdk@1c75e97108)

Since Apple JDK was present only up till JDK 6, we no longer require the
special case for MacOS since JDK 8.

Thus, making the test argument value `specialMacTreatment = false` for
these two tests, and renaming the corresponding goldenfiles from
"-nonmac.pass" suffix to ".pass" suffix.

Signed-off-by: Siddharth Srinivasan <[email protected]>
…s_javahints

Fix MacOS tests failures in java.hints
 - simple script to check if all commits have proper email addresses
 - looks for empty or single word subject messages
 - can warn about missing blank line after subject (disabled)
 - logs errors/warnings to the github actions summary page, fails on
   error

paperwork job:

 - the paperwork job will now no longer prevent the github workspace
   from being cleaned up on failure
@mbien mbien force-pushed the clone-build branch 2 times, most recently from b6c9ffc to 341da28 Compare November 3, 2024 23:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.