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

[cnd] merge master into cnd #6439

Merged
merged 1,990 commits into from
Sep 26, 2023
Merged

[cnd] merge master into cnd #6439

merged 1,990 commits into from
Sep 26, 2023

Conversation

bitrunner
Copy link

This merges master into the cnd branch bringing it up to date with the changes since May 2022.

1 conflict with nbbuild/cluster.properties was resolved by taking the changes from master.

1 build error in cnd/cnd.ui/src/org/netbeans/modules/cnd/loaders/QtUiDataLoader.java due to changes to org.openide.util.MapFormat API was resolved by doing a type conversion.

matthiasblaesing and others added 30 commits July 7, 2023 21:58
…port

Fix wrong javac tree access for module information in modular maven project
Update JDK version requirement warning
* Made HCLTokenId more detailed, useful for indenter/formater

* Added basic line indenting to HCL languages

* Fix IOOB exception when trying to parse '"' as identifier

* Indent is based on the previous line from now.

* Enable Flyweight tokens for HCL Lexer

* Added some tests for the indenter.
25347df introduces a new unittest,
that proved to be unstable in CI/CD pipeline. The failure can be
reproduced after several tries. Adding 

workDir.getFileSystem().refresh(true);

seems to fix the issue. Hypothesis:

The new module files are written outside the filesystem infrastructure
of NetBeans and the cache of the javac integration does not pick up the
module files created by the javac-Tool.

Apart from the fix, style of the code was improved.
…with-cast

Fix `instanceof` expression with casting apache#5933
- Update the version to PHPUnit 10
- Remove the unavailable URL from readme.html
- Add setting for code coverage to readme.html
- Fix the phpunit.xml (the `filter` element is available no longer. Instead, use the `coverage` element.)
- See: https://docs.phpunit.de/en/10.1/configuration.html (https://phpunit.de/documentation.html)
- apache#6075
- Fix the scanner: Add the comments as `Comment.Type.TYPE_VARTYPE` in the following example
- Add unit tests

Example:
```php
/** @var ?User $user */
$user->getId();

/* @var $user2 ?User */
$user2->getId();
```
…ependencies

Avoid needless dependency on LineDocument
…port_2

Fix unstable unittest org.netbeans.modules.maven.ModuleInfoSupportTest
- apache#6075
- Check types without nullable type prefix(`?`)
- Add unit tests
…or-phpunit

Update the PHP sample project for PHPUnit
…-nullable-type

Fix "Fix Imports" when the vardoc has nullable types
- apache#6119
- Check whether the class name has `$`
- Add unit tests
…bstract-instantiation-hint

Fix incorrect AbstractClassInstantiationHintError apache#6119
Fix tab closing when trying to select partly clipped tab.
…ception when calling DatabaseConnection.showConnectionDialog from the EDT.
Provide Lambda expression option in Listener generation style
- apache#5578

Example:
```php
namespace NS1;

class A
{
    public function __construct()
    {
        declare (ticks=1) {
        }
        $test = new B();
    }
}
```

Before:
```php
namespace NS1;

class A
{
    public function __construct()
    {
use NS2\B;
        declare (ticks=1) {
        }
        $test = new B();
    }
}
```

After:
```php
namespace NS1;

use NS2\B;

class A
{
    public function __construct()
    {
        declare (ticks=1) {
        }
        $test = new B();
    }
}
```
- apache#5578

Example:
```php
<?php
namespace NS1;

declare(ticks=1) {
    $test = 1;
}
declare(ticks=2) {
    $test = 1;
}
class DeclareTest1 {
    public function test() {
        declare(ticks=2) {

        }
        $test = new \NS2\DeclareTest2;
    }
}
```

Before:
```php
<?php
namespace NS1;

declare(ticks=1) {
    $test = 1;
}
use NS2\DeclareTest2;
declare(ticks=2) {
    $test = 1;
}
class DeclareTest1 {
    public function test() {
        declare(ticks=2) {

        }
        $test = new DeclareTest2;
    }
}
```

After:
```php
<?php
namespace NS1;

declare(ticks=1) {
    $test = 1;
}
declare(ticks=2) {
    $test = 1;
}

use NS2\DeclareTest2;

class DeclareTest1 {
    public function test() {
        declare(ticks=2) {

        }
        $test = new DeclareTest2;
    }
}
```
… for use statements

- apache#5681
- Avoid changing the file state if the created string is the same as the
  existing use statements
neilcsmith-net and others added 18 commits September 7, 2023 11:59
RustLanguageLexer: do not throw exceptions.
… and continue, rather than crash the whole processing.
Handle errors in annotation processor initialization
…ls-fix

LSP: Fixed problem with workspace/symbols called before first JavacCompiler created.
…e-participant

Micronaut's lifecycle participant changed FQN in Micronaut 4.x
…usion

Workaround: Avoid reporting BOMs among dependencies to prevent dependency graphs explosion
@vieiro
Copy link
Contributor

vieiro commented Sep 13, 2023

Hi @bitrunner ,

Thanks for your interest in the cnd branch!

I think it's better to discuss in the mailing list how to proceed with the cnd branch. Maybe we want to integrate it with master and disable the cluster by default, much like the rust branch is being moved forward.

Thanks,
Antonio

@mbien mbien added the C/C++ label Sep 14, 2023
@bitrunner
Copy link
Author

As someone new to netbeans development, I don't feel comfortable offering an opinion on or injecting myself into big picture decisions. I'm just a long time CND user with enough free time and motivation right now to try to help make CND great again. Let's make some forward progress happen.

@vieiro
Copy link
Contributor

vieiro commented Sep 16, 2023

As someone new to netbeans development, I don't feel comfortable offering an opinion on or injecting myself into big picture decisions. I'm just a long time CND user with enough free time and motivation right now to try to help make CND great again. Let's make some forward progress happen.

Hi @bitrunner ,

I see. We'll, I think we'll have to decide how to move cnd forward. There're many different possibilities. Let's see what people say in the mailing list and then we'll move things forward.

"long time CND user" opinions are indeed welcome in the mailing list, even more if those users are motivated :-).

@vieiro vieiro added the ci:all-tests [ci] enable all tests label Sep 24, 2023
@vieiro
Copy link
Contributor

vieiro commented Sep 25, 2023

Hi @matthiasblaesing , this looks fine to me. Mind taking a second look?

Copy link
Contributor

@matthiasblaesing matthiasblaesing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks sane to me. I had a look at the merge and the decisions look sane to me. @bitrunner @vieiro thank you for continued working on this.

@vieiro vieiro merged commit 6b5a36b into apache:cnd Sep 26, 2023
34 of 35 checks passed
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.