Skip to content

PE-70724: Remove Qlty from react-native-picker-select pipeline#741

Merged
gumaestri merged 1 commit intomasterfrom
PE-70274
Mar 30, 2026
Merged

PE-70724: Remove Qlty from react-native-picker-select pipeline#741
gumaestri merged 1 commit intomasterfrom
PE-70274

Conversation

@gumaestri
Copy link
Copy Markdown
Contributor

Summary

  • Deletes the .qlty/ configuration directory
  • Removes the "Upload coverage to Qlty" step from .github/workflows/build.yml

Note

The QLTY_COVERAGE_TOKEN GitHub secret should also be removed manually from repo settings if present.

Test plan

  • CI pipeline runs successfully with no regressions

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 27, 2026

📝 Walkthrough

Walkthrough

This pull request removes Qlty integration from the project by deleting configuration files (.qlty/qlty.toml and .yamllint.yaml), clearing the .qlty/.gitignore rules, and removing the coverage upload step from the CI/CD build workflow.

Changes

Cohort / File(s) Summary
Qlty Configuration Removal
.qlty/qlty.toml, .qlty/configs/.yamllint.yaml
Removed complete Qlty configuration including code smell detection settings, patterns, and Yamllint rules.
Gitignore Cleanup
.qlty/.gitignore
Removed all ignore/unignore rules from the .qlty/ directory gitignore file.
CI/CD Workflow
.github/workflows/build.yml
Removed the Qlty coverage upload step (qltysh/qlty-action/coverage@v2) from the build workflow, eliminating external coverage publication after tests.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description check ✅ Passed The description is directly related to the changeset, providing a clear summary of the two main changes (deleting .qlty/ directory and removing the coverage upload step) and noting the manual cleanup needed.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title clearly and concisely summarizes the main change: removing Qlty from the CI/CD pipeline by deleting configuration files and workflow steps.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch PE-70274

Comment @coderabbitai help to get the list of available commands and usage tips.

@gumaestri gumaestri changed the title conf(qlty): Remove Qlty from react-native-picker-select pipeline PE-70724: Remove Qlty from react-native-picker-select pipeline Mar 27, 2026
@gumaestri gumaestri merged commit b58e5d8 into master Mar 30, 2026
8 checks passed
@gumaestri gumaestri deleted the PE-70274 branch March 30, 2026 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants