Skip to content

theme: screenshot added#759

Merged
akinomyoga merged 3 commits intoohmybash:masterfrom
debarkak:master
May 4, 2026
Merged

theme: screenshot added#759
akinomyoga merged 3 commits intoohmybash:masterfrom
debarkak:master

Conversation

@debarkak
Copy link
Copy Markdown
Contributor

@debarkak debarkak commented May 2, 2026

this PR provides the screenshot for the garo theme to be referenced on the wiki.

  • taken on fedora 43 with gnome-terminal 3.5.6

@qodo-free-for-open-source-projects
Copy link
Copy Markdown

qodo-free-for-open-source-projects Bot commented May 2, 2026

Review Summary by Qodo

(Agentic_describe updated until commit ce6d6f1)

Add Garo theme screenshot for wiki documentation

📝 Documentation

Grey Divider

Walkthroughs

Description
  * Adds screenshot for the Garo theme to be referenced on the wiki
  * Screenshot captured on Fedora 43 with GNOME Terminal 3.5.6
Diagram
flowchart LR
  A["Garo Theme"] -- "screenshot added" --> B["Wiki Documentation"]
Loading

Grey Divider

File Changes

Grey Divider

Qodo Logo

@qodo-free-for-open-source-projects
Copy link
Copy Markdown

Code Review by Qodo

Grey Divider

No Changes in PR

Qodo reviewed your PR and found no changes in the code

Grey Divider

Qodo Logo

@akinomyoga
Copy link
Copy Markdown
Contributor

Could you check New Theme - CONTRIBUTING for the image size? THEMES.md also needs to be updated.

@akinomyoga
Copy link
Copy Markdown
Contributor

Also, the margin can be removed.

@debarkak
Copy link
Copy Markdown
Contributor Author

debarkak commented May 3, 2026

@akinomyoga ok i will make a new pull request with the fixes, i will close this one.

@debarkak debarkak closed this May 3, 2026
@akinomyoga
Copy link
Copy Markdown
Contributor

akinomyoga commented May 3, 2026

Please do not create a new PR. We should continue the discussion in the same PR to avoid scattering the relevant discussions in different places. When you push the new commits to the head branch (dabarkak:master this time), the new changes are automatically reflected in this PR page.

@akinomyoga akinomyoga reopened this May 3, 2026
@qodo-free-for-open-source-projects
Copy link
Copy Markdown

Code Review by Qodo

Grey Divider

No Changes in PR

Qodo reviewed your PR and found no changes in the code

Grey Divider

Qodo Logo

@debarkak
Copy link
Copy Markdown
Contributor Author

debarkak commented May 3, 2026

Please do not check

I didn't understand, what do you mean by this?

@akinomyoga
Copy link
Copy Markdown
Contributor

I've mistakenly posted the comment before I finish editing it. I started to input some text, but then I thought I could reopen the PR before writing the comment. Then, I pressed the Reopen button of the GitHub interface, which somehow posted the text that I didn't finish writing at the same time.

@akinomyoga
Copy link
Copy Markdown
Contributor

I've already edited the original comment, which was one minute after you asked the question. Could you check the edited comment?

@debarkak
Copy link
Copy Markdown
Contributor Author

debarkak commented May 3, 2026

@akinomyoga i have edited the THEMES.md file along with the screenshot, i couldn't get exactly what the CONTRIBUTING NEW THEME specified but i think i did enough close.

@akinomyoga
Copy link
Copy Markdown
Contributor

Thank you. The updated PR satisfies the requests in CONTRIBUTING. I'll merge the PR.

@akinomyoga akinomyoga merged commit ae44713 into ohmybash:master May 4, 2026
4 checks passed
@akinomyoga
Copy link
Copy Markdown
Contributor

I've merged. Thank you.

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