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

docs(vector): Clarify suggested usage #76

Merged
merged 1 commit into from
Oct 1, 2021

Conversation

Yasumoto
Copy link
Contributor

@Yasumoto Yasumoto commented Oct 1, 2021

These charts used to be hosted within the main repo, and #24 added a combined chart that looks to be the future. This adds a (likely short-term) pointer in the README to help guide folks to the right path.

Maybe most importantly, please consider this a strawman since I'm not entirely certain this is the chart y'all recommend folks use!

This is related to vectordotdev/vector#9396

@Yasumoto Yasumoto changed the title enhancement(docs): Clarify suggested usage docs(vector): Clarify suggested usage Oct 1, 2021
These charts used to be hosted within the main repo, and vectordotdev#24 added a combined chart that looks to be the future. This adds a (likely short-term) pointer in the README to help guide folks to the right path.

Signed-off-by: Joe Smith <yasumoto7@gmail.com>
@spencergilbert
Copy link
Contributor

Thanks! I was actually holding off on the messaging until I moved the vector chart out of pre-release (hopefully today/early next week) - but the way you phrased it makes sense to me right now 😄

@spencergilbert spencergilbert merged commit 0555fb0 into vectordotdev:develop Oct 1, 2021
@Yasumoto Yasumoto deleted the patch-1 branch October 1, 2021 20:19
@Yasumoto
Copy link
Contributor Author

Yasumoto commented Oct 1, 2021

Totally makes sense! As I started to look into the repo and recent changes I figured that was the case 👍🏼

Love all the changes you're making, it's going to be great!

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.

None yet

2 participants