Skip to content
This repository has been archived by the owner on Jun 8, 2021. It is now read-only.

Include metadata.json #10

Open
trieloff opened this issue May 3, 2021 · 4 comments
Open

Include metadata.json #10

trieloff opened this issue May 3, 2021 · 4 comments
Labels
enhancement New feature or request

Comments

@trieloff
Copy link
Contributor

trieloff commented May 3, 2021

From #2 (review)

@davidnuescheler ideally, we could also consider the existence of a metadata.json as config. i.e. just add a:

"metadata": "/metadata.json`

entry. so we don't need to fetch a 404 if there is none.

@tripodsan
Copy link
Contributor

I suggest:

"metadata": {
  "path": "/metadata.json"
}

@trieloff
Copy link
Contributor Author

trieloff commented May 3, 2021

Why not inline it?

@rofe
Copy link
Contributor

rofe commented May 3, 2021

Because it can potentially be gigantic...

@tripodsan
Copy link
Contributor

Why not inline it?

and because it's content and changes often. so we don't have triggers to copy it to the code-bus

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

No branches or pull requests

3 participants