Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

adds create #21

Merged
merged 3 commits into from
Jul 24, 2023
Merged

adds create #21

merged 3 commits into from
Jul 24, 2023

Conversation

afflom
Copy link
Contributor

@afflom afflom commented Jul 17, 2023

Docs: Create

Type of change

Doc/drawing

Signed-off-by: Alex Flom <[email protected]>
@jpower432
Copy link
Member

@afflom I am thinking that might may need to split up the diagrams. One showing the tasks that trestle-bot performs then another document to show possible workflows. Some ideas I am considering:

  • Default path (Assemble + Regenerate + Push Commit back to branch)
  • Regenerate only
  • Check Only
  • New imports/creation + trestlebot

Thoughts?

@afflom
Copy link
Contributor Author

afflom commented Jul 19, 2023

@afflom I am thinking that might may need to split up the diagrams. One showing the tasks that trestle-bot performs then another document to show possible workflows. Some ideas I am considering:

  • Default path (Assemble + Regenerate + Push Commit back to branch)
  • Regenerate only
  • Check Only
  • New imports/creation + trestlebot

Thoughts?

Would these share a container view and be differentiated in different component views?

@jpower432
Copy link
Member

@afflom Changing my feedback due to the updates given in #24. It would probably be more useful to provide snippets to show how each workflow could be done instead of working them into a diagram. The C4 model diagram code perhaps just be a component and code view of trestle-bot. Thoughts?

jpower432 and others added 2 commits July 21, 2023 16:26
docs: adds c4 architecture drawing for context,containers, and component
@jpower432 jpower432 merged commit 992553f into RedHatProductSecurity:main Jul 24, 2023
6 checks passed
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