Skip to content

correct db comment - #95

Merged
rm3l merged 1 commit into
redhat-developer:mainfrom
chrira:docs-updates
Sep 5, 2025
Merged

correct db comment#95
rm3l merged 1 commit into
redhat-developer:mainfrom
chrira:docs-updates

Conversation

@chrira

@chrira chrira commented Sep 4, 2025

Copy link
Copy Markdown
Contributor

Description

Updated comment in compose.yaml.

  • Fixed spelling
  • Added alternative config file

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

PR acceptance criteria

  • [/] Tests
  • [/] Documentation

How to test changes / Special notes to the reviewer

Only changes in comments.

Summary by Sourcery

Update compose.yaml comment to correct spelling and clarify database configuration file options

Documentation:

  • Fix spelling of 'comment out' in the database service comment
  • Include both app-config.yaml and app-config.local.yaml as alternative config file options in the comment

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey there - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@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.

Thanks @chrira !!

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label Sep 5, 2025
@rm3l
rm3l merged commit 35e3aa5 into redhat-developer:main Sep 5, 2025
25 checks passed
@chrira
chrira deleted the docs-updates branch September 8, 2025 04:28
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.

2 participants