.github/workflows/test.yml (3)

531-531: Remove trailing whitespace
This line has trailing spaces, which will cause a YAML lint error.

🧰 Tools

🪛 YAMLlint (1.37.1)

[error] 531-531: trailing spaces

(trailing-spaces)

---

539-539: Limit secret scope
Using secrets: inherit exposes all repository secrets to the reusable workflow. For better security and auditability, consider passing only the specific secrets required by this job.

---

540-542: Remove extra blank lines
There are multiple blank lines here; reduce to a single blank line to satisfy YAML lint rules.

🧰 Tools

🪛 YAMLlint (1.37.1)

[warning] 541-541: too many blank lines

(3 > 0) (empty-lines)

Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro

.github/workflows/test.yml (1)

530-533: Ensure workflow triggers on label addition events
The if condition checks PR labels, but on: pull_request is limited to opened, synchronize, and reopened. Label additions won’t retrigger the job.

Consider adding types: [labeled] under on: pull_request or switching to pull_request_target with types: [labeled].

🧰 Tools

🪛 YAMLlint (1.37.1)

[error] 531-531: trailing spaces

(trailing-spaces)

.github/workflows/test.yml (2)

531-531: Remove trailing whitespace
Line 531 has trailing spaces.

🧰 Tools

🪛 YAMLlint (1.37.1)

[error] 531-531: trailing spaces

(trailing-spaces)

---

540-543: Remove extra blank lines and trailing spaces
There are multiple blank lines (and trailing spaces) before the end of the job definition.

🧰 Tools

🪛 YAMLlint (1.37.1)

[error] 540-540: trailing spaces

(trailing-spaces)

---

[warning] 543-543: too many blank lines

(3 > 0) (empty-lines)

Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro

.github/workflows/test.yml (2)

535-536: 🛠️ Refactor suggestion

Make publish and prerelease flags conditional

Hardcoding both flags triggers prereleases on all pushes (e.g., main or release branches). Restrict publishing to pushes and prereleasing to PRs:

-with:
-  publish: true
-  prerelease: true
+with:
+  publish: ${{ github.event_name == 'push' }}
+  prerelease: ${{ github.event_name == 'pull_request' }}

---

530-532: 🛠️ Refactor suggestion

Ensure release job triggers on label additions

The workflow only listens to pull_request events for opened/synchronize/reopened. Adding release/feature later won’t retrigger the job. Include labeled in on.pull_request.types:

on:
  pull_request:
-    types: [opened, synchronize, reopened]
+    types: [opened, synchronize, reopened, labeled]

🧰 Tools

🪛 YAMLlint (1.37.1)

[error] 531-531: trailing spaces

(trailing-spaces)

.github/workflows/test.yml (4)

531-531: Remove trailing spaces

Lint flagged trailing whitespace here. Please trim the line.

🧰 Tools

🪛 YAMLlint (1.37.1)

[error] 531-531: trailing spaces

(trailing-spaces)

---

538-538: Scope environment input by trigger

environment: 'feature-releases' applies to every run. If full releases should use a different environment (e.g., production), consider:

environment: ${{ github.event_name == 'pull_request' && 'feature-releases' || 'production' }}

---

533-533: Pin the external workflow reference

Using @main can introduce breaking changes when that repo updates. Pin to a specific tag or SHA for reproducible releases.

---

542-542: Eliminate extra blank lines

There are several consecutive blank lines here. Collapse to a single blank line for clarity.

🧰 Tools

🪛 YAMLlint (1.37.1)

[warning] 542-542: too many blank lines

(3 > 0) (empty-lines)

Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro

.github/settings.yml (2)

14-14: Remove trailing whitespace
Trailing spaces on blank lines can cause YAML lint failures. Please delete the extra spaces on line 14.

🧰 Tools

🪛 YAMLlint (1.37.1)

[error] 14-14: trailing spaces

(trailing-spaces)

---

25-25: Remove extra blank line
There’s an extra blank line after the labels section which triggers a YAML lint warning. You can collapse to a single blank line between sections.

🧰 Tools

🪛 YAMLlint (1.37.1)

[warning] 25-25: too many blank lines

(1 > 0) (empty-lines)

Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro

.github/workflows/test.yml (2)

3-4: Trigger on label addition
Labeling a PR won’t rerun this workflow since pull_request.types excludes labeled.
Add the labeled type:

 on:
   pull_request:
-    types: [opened, synchronize, reopened]
+    types: [opened, synchronize, reopened, labeled]

---

535-536: Make publish/prerelease conditional
Hardcoding both to true will publish prereleases on push events.
Use expressions instead:

-      publish: true
-      prerelease: true
+      publish: ${{ github.event_name == 'push' }}
+      prerelease: ${{ github.event_name == 'pull_request' && contains(github.event.pull_request.labels.*.name, 'release/feature') }}

.github/workflows/test.yml (2)

531-531: Trim trailing whitespace
YAML lint flags a trailing space here.

-if: |␣
+if: |

🧰 Tools

🪛 YAMLlint (1.37.1)

[error] 531-531: trailing spaces

(trailing-spaces)

---

538-538: Conditional environment
All releases (even prod pushes) target feature-releases. Consider setting environment based on event or label to keep prod and feature channels separate.

Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro