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: minor edit to the who supports ibis doc #7896

Merged
merged 3 commits into from Jan 4, 2024

Conversation

lostmygithubaccount
Copy link
Member

Description of changes

reading the doc back, making some very minor edits

Issues closed

n/a

Copy link
Contributor

github-actions bot commented Jan 4, 2024

ACTION NEEDED

Ibis follows the Conventional Commits specification for release automation.

The PR title and description are used as the merge commit message.

Please update your PR title and description to match the specification.

@lostmygithubaccount lostmygithubaccount changed the title minor edit to the who supports ibis doc docs: minor edit to the who supports ibis doc Jan 4, 2024
docs/concepts/who.qmd Outdated Show resolved Hide resolved
Copy link
Member

@cpcloud cpcloud left a comment

Choose a reason for hiding this comment

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

The updated Other companies section seems wrong :)

@cpcloud cpcloud added analytics Higher-level data analysis API related issues or PRs docs Documentation related issues or PRs and removed analytics Higher-level data analysis API related issues or PRs labels Jan 4, 2024
@cpcloud cpcloud added this to the 8.0 milestone Jan 4, 2024
@lostmygithubaccount
Copy link
Member Author

and to answer you question -- trying to note that other companies also contribute to Ibis, not just build things on top of it

@cpcloud
Copy link
Member

cpcloud commented Jan 4, 2024

and to answer you question -- trying to note that other companies also contribute to Ibis, not just build things on top of it

Makes sense.

My main concern was with this fragment:

Various tools built on top of it.

@cpcloud cpcloud merged commit d5a0779 into ibis-project:main Jan 4, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation related issues or PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants