Skip to content

Remove parquet-factory linter copy#77

Merged
joselsegura merged 1 commit into
RedHatInsights:masterfrom
joselsegura:remove_parquet_factory_linters
Apr 20, 2026
Merged

Remove parquet-factory linter copy#77
joselsegura merged 1 commit into
RedHatInsights:masterfrom
joselsegura:remove_parquet_factory_linters

Conversation

@joselsegura
Copy link
Copy Markdown
Contributor

Description

Parquet Factory implements its own logic for linter workflow, as it needs to first build a set of mocks

Type of change

  • Configuration update

Testing steps

N/A

@joselsegura joselsegura requested a review from a team as a code owner April 16, 2026 15:54
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 16, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 69d46c91-5f88-408d-bd60-84558dc41d83

📥 Commits

Reviewing files that changed from the base of the PR and between fd3f81e and 2a312ee.

📒 Files selected for processing (1)
  • .github/sync.yml
💤 Files with no reviewable changes (1)
  • .github/sync.yml

Summary by CodeRabbit

Release Notes

  • Chores
    • Removed automated linter workflow synchronization rule from GitHub workflow configuration.

Walkthrough

A sync rule mapping in the parquet-factory configuration was removed that previously synchronized a linters workflow example file to the GitHub workflows directory. All other sync mappings remain unchanged.

Changes

Cohort / File(s) Summary
Sync Configuration
.github/sync.yml
Removed the sync rule that copied workflows_examples/linters.yaml to .github/workflows/linters.yaml from the RedHatInsights/parquet-factory mapping.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: removing the linter copy rule for parquet-factory from the sync configuration.
Description check ✅ Passed The description is related to the changeset, explaining why the linter configuration is being removed and categorizing it as a configuration update.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@joselsegura joselsegura merged commit b13139a into RedHatInsights:master Apr 20, 2026
3 checks passed
@joselsegura joselsegura deleted the remove_parquet_factory_linters branch April 20, 2026 06:07
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.

3 participants