Skip to content

Update .gitignore to include additional files - #121

Merged
rm3l merged 4 commits into
redhat-developer:mainfrom
benwilcock:patch-1
Nov 21, 2025
Merged

Update .gitignore to include additional files#121
rm3l merged 4 commits into
redhat-developer:mainfrom
benwilcock:patch-1

Conversation

@benwilcock

Copy link
Copy Markdown
Contributor

Three suggestions in the PR:

  1. allow the user to have their own scripts in the root folder
  2. allow the user to add their own override for the compose file
  3. allow the user to add .bak etc. to common override files

For me, these would reduce the conflicts and prevent the need to stash these files prior to running git pull on the repo to keep up to date with the latest changes

Description

Which issue(s) does this PR fix or relate to

  • Fixes #issue_number

PR acceptance criteria

  • Tests updated and passing
  • Documentation updated
  • Built-in TechDocs updated if needed. Note that TechDocs changes may need to be reviewed by a Product Manager and/or Architect to ensure content accuracy, clarity, and alignment with user needs.

How to test changes / Special notes to the reviewer

Three suggestions in the PR:

1. allow the user to have their own scripts in the root folder
2. allow the user to add their own override for the compose file
3. allow the user to add `.bak` etc. to common override files 

For me, these would reduce the conflicts and prevent the need to stash these files prior to running `git pull` on the repo to keep up to date with the latest changes
Making sure that `.bak` etc. can also be added to the compose override to prevent the need for stashing local backups.
@rm3l

rm3l commented Oct 31, 2025

Copy link
Copy Markdown
Member

/review

Comment thread .gitignore Outdated
@zdrapela

zdrapela commented Nov 7, 2025

Copy link
Copy Markdown
Member

/review

1 similar comment
@zdrapela

zdrapela commented Nov 7, 2025

Copy link
Copy Markdown
Member

/review

@redhat-developer redhat-developer deleted a comment from qodo-code-review Bot Nov 7, 2025
@redhat-developer redhat-developer deleted a comment from rhdh-qodo-merge Bot Nov 7, 2025
@zdrapela

zdrapela commented Nov 7, 2025

Copy link
Copy Markdown
Member

/review

benwilcock and others added 2 commits November 21, 2025 17:15
Co-authored-by: Armel Soro <armel@rm3l.org>

@rm3l rm3l left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label Nov 21, 2025
@rm3l
rm3l merged commit 0fa3187 into redhat-developer:main Nov 21, 2025
49 of 50 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants