Skip to content

Added: .claude and .cursor personal AI config dir to .gitignore#1862

Merged
openshift-merge-bot[bot] merged 1 commit intomainfrom
fix/add-ai-config-to-gitignore
Mar 25, 2026
Merged

Added: .claude and .cursor personal AI config dir to .gitignore#1862
openshift-merge-bot[bot] merged 1 commit intomainfrom
fix/add-ai-config-to-gitignore

Conversation

@oliashish
Copy link
Contributor

@oliashish oliashish commented Mar 24, 2026

User should not commit their personal AI config under .claude or .cursor directory.

This is already added to edpm-ansible and ci-framework repos. I don't know the reasoning to not do it in this project.

Please let me know if this is already discussed and is not needed. I couldn't find it in slack or spaces.

UPDATE: I checked when did this patch was done on edpm-ansible and it is recent (3 weeks ago)

@oliashish oliashish requested review from abays, rabi and slagle March 24, 2026 20:47
@openshift-ci openshift-ci bot requested a review from stuggi March 24, 2026 20:47
@github-actions
Copy link

github-actions bot commented Mar 24, 2026

OpenStackControlPlane CRD Size Report

Metric Value
CRD JSON size 322136 bytes (315KB)
Base branch size 322136 bytes
Change +0.00%
Status yellow — growing
Threshold reference
Color Range Meaning
🟢 green < 300KB Comfortable
🟡 yellow 300–400KB Growing
🟠 orange 400–750KB Concerning
🔴 red > 750KB Approaching 1.5MB etcd limit (cut in half to allow space for update)

@oliashish oliashish force-pushed the fix/add-ai-config-to-gitignore branch from 5eb09b5 to f987509 Compare March 24, 2026 20:50
@oliashish oliashish changed the title Added: claude and cursor personal config dir to .gitignore Added: .claude and c.ursor personal config dir to .gitignore Mar 24, 2026
@oliashish oliashish changed the title Added: .claude and c.ursor personal config dir to .gitignore Added: .claude and .cursor personal AI config dir to .gitignore Mar 24, 2026
Copy link
Contributor

@slagle slagle left a comment

Choose a reason for hiding this comment

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

+2

Copy link
Contributor

@stuggi stuggi left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 25, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: oliashish, stuggi

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot bot merged commit ab21714 into main Mar 25, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants