Skip to content

Add pictures to documentation - #1055

Merged
softwarefactory-project-zuul[bot] merged 1 commit into
packit:mainfrom
betulependule:add-pictures
Aug 19, 2025
Merged

Add pictures to documentation#1055
softwarefactory-project-zuul[bot] merged 1 commit into
packit:mainfrom
betulependule:add-pictures

Conversation

@betulependule

Copy link
Copy Markdown
Contributor

I've added a couple of pictures to the documentation (from old presentations, and screenshots of public repos) in places I thought might fit. I would personally remove most of them. Please let me know if there is something that would be better removed or moved elsewhere.

TODO:

  • Remove unnecessary pictures (if any)

Fixes #2638

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@betulependule
betulependule marked this pull request as ready for review August 18, 2025 07:49

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

I love the process-diagram picture! Super helpful.
And also all the others pictures you have added can be really helpful.
Thanks a lot!

Comment thread posts/aws-and-image-builder/index.md Outdated
@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@betulependule

Copy link
Copy Markdown
Contributor Author

Should be fixed now, I hope

@majamassarini
majamassarini dismissed their stale review August 19, 2025 08:04

Already fixed in a different way

@majamassarini majamassarini 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!

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

looks great!

@betulependule betulependule added the mergeit Merge via Zuul label Aug 19, 2025
@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

Build succeeded (gate pipeline).
https://softwarefactory-project.io/zuul/t/packit-service/buildset/b18ed4dd57524f7db0d9e61d8ce57a91

✔️ pre-commit SUCCESS in 1m 13s

@softwarefactory-project-zuul
softwarefactory-project-zuul Bot merged commit 403d340 into packit:main Aug 19, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mergeit Merge via Zuul

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add pictures to documentation

4 participants