opensearch-project/OpenSearch

Ensure that warnings as errors are enabled for CI checks on main

Open

#9.150 geöffnet am 7. Aug. 2023

Auf GitHub ansehen
 (6 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Java (1.505 Forks)batch import
BuildBuild Libraries & InterfacesCIbuggood first issue

Repository-Metriken

Stars
 (8.123 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 5T 9h) (266 gemergte PRs in 30 T)

Beschreibung

Following the merge of https://github.com/opensearch-project/OpenSearch/pull/9120, publication of maven snapshots started failing due to a compilation warning. On the backport of that change to 2.x, the @SuppressWarnings("unchecked") annotation was added to the method with the compilation warning to ensure that the backport PR passed CI checks.

It looks like there may be a difference between how CI checks are performed on main vs 2.x since it was required to add the annotation in 2.x, but not on main. I'm filing an issue to investigate why there is a difference in the checks between the 2 branches and ensure that these checks are running on the main branch as well as 2.x.

Issue discussed in PR to resolve the maven snapshot publication here: https://github.com/opensearch-project/OpenSearch/pull/9149

Contributor Guide