Skip to content

theme: screenshot added#759

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

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

Conversation

@debarkak

@debarkak debarkak commented May 2, 2026

Copy link
Copy Markdown
Contributor

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

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

Copy link
Copy Markdown

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

debarkak commented May 3, 2026

Copy link
Copy Markdown
Contributor Author

@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

akinomyoga commented May 3, 2026

Copy link
Copy Markdown
Contributor

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

debarkak commented May 3, 2026

Copy link
Copy Markdown
Contributor Author

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

debarkak commented May 3, 2026

Copy link
Copy Markdown
Contributor Author

@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