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

feat(core): added ability to specify dependencies for catalogs #1033

Merged
merged 4 commits into from
Dec 18, 2024

Conversation

boyney123
Copy link
Collaborator

From #930, added the ability to create catalog dependencies. These are resources that a team may need for there catalog, but not necessarily own themself.

By specifying a dependency EventCatalog will create these resource types for the teams allowing them to use EventCatalog with resources that are external.

Copy link

changeset-bot bot commented Dec 16, 2024

🦋 Changeset detected

Latest commit: 83e483d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@eventcatalog/core Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

@carlosallexandre carlosallexandre left a comment

Choose a reason for hiding this comment

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

Wouldn't be better if the owner of the catalog create the markdown themselves instead of specifying on the config file?

@boyney123
Copy link
Collaborator Author

"Wouldn't be better if the owner of the catalog create the markdown themselves instead of specifying on the config file?"

Yeah good question. This is to handle a usecase around multi-catalog configurations into one master one.

Team A have a catalog in their repo, and they consume an Event from Team B, today they have to document this, but they do not own it

This gives the users the ability to "mock" out events that they may need. in this case Team A would mock the event, they know they are using, but don't own the contract or docs for it.... (if that makes sense)

@boyney123 boyney123 merged commit 972478d into main Dec 18, 2024
4 checks passed
@boyney123 boyney123 deleted the added-new-dependencies-config-options branch December 18, 2024 11:08
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