Skip to content

Conversation

@betulependule
Copy link
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
Contributor

@softwarefactory-project-zuul
Copy link
Contributor

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

@majamassarini majamassarini left a comment

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!

@softwarefactory-project-zuul
Copy link
Contributor

@softwarefactory-project-zuul
Copy link
Contributor

@softwarefactory-project-zuul
Copy link
Contributor

@betulependule
Copy link
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

Copy link
Member

@majamassarini majamassarini left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Member

@lbarcziova lbarcziova left a comment

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