Skip to content
This repository has been archived by the owner on Jul 23, 2024. It is now read-only.

Create catalog-info.yaml #459

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

stan-dot
Copy link

@stan-dot stan-dot commented Jun 6, 2024

No description provided.

@garryod
Copy link
Member

garryod commented Jun 10, 2024

Components should map to single libraries or executables, not a single component for a collection of services

@stan-dot
Copy link
Author

ok, so I'll make it into a top-level system and then components inside.

@stan-dot
Copy link
Author

the errors do not come from the contents of this PR

@garryod
Copy link
Member

garryod commented Jun 11, 2024

There are multiple backend components, the Components created should reflect this

@stan-dot
Copy link
Author

Ok I will make it reflect this

@stan-dot
Copy link
Author

how low level would you like this @garryod ? should I treat chimp-chomp as one service, or split it very low-level into worker, controller etc?

@garryod
Copy link
Member

garryod commented Jul 1, 2024

how low level would you like this @garryod ? should I treat chimp-chomp as one service, or split it very low-level into worker, controller etc?

All of them are Components, the bins are services whilst the libs are libraries

@stan-dot
Copy link
Author

stan-dot commented Jul 1, 2024

thanks for the clarification

@stan-dot
Copy link
Author

stan-dot commented Jul 3, 2024

I was trying to replicate the error from the lint and docs

 error: linker `cc` not found
  |
 = note: No such file or directory (os error 2)

this does not work -

cargo doc --workspace --no-deps --all-features --document-private-items --manifest-path ./Car
go.toml
the same error happens when I run cargo clippy

@stan-dot
Copy link
Author

I think if this pipeline is re-run it might pick up the updates from the soakdb which caused the error. I cannot trigger the pipeline though @garryod

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants