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

Add reference to #comet channel in Arrow Rust Discord server #11637

Merged
merged 2 commits into from
Jul 25, 2024

Conversation

ajmarcus
Copy link
Contributor

Which issue does this PR close?

None (documentation only change)

Rationale for this change

I wasn't sure where discussion about comet happens.

What changes are included in this PR?

Documentation change to refer to the #comet channel in the Arrow Rust Discord server

Are these changes tested?

No - documentation change only

Are there any user-facing changes?

Yes. Documentation change.

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jul 24, 2024
@@ -38,7 +38,7 @@ meet other contributors and get guidance on where to contribute. It is important
decisions are made fully in the open, on GitHub.

Most of us use the `#datafusion` and `#arrow-rust` channels in the [ASF Slack
workspace](https://s.apache.org/slack-invite) and the [Arrow Rust Discord
workspace](https://s.apache.org/slack-invite) and the `#comet` channel in the [Arrow Rust Discord
server](https://discord.gg/Qw5gKqHxUM) for discussions.
Copy link
Member

@andygrove andygrove Jul 24, 2024

Choose a reason for hiding this comment

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

I think it is a good idea to make it easier to find subproject-specific channels, so we should list all of them.

In Slack we have:

Screenshot from 2024-07-24 10-18-04

In Discord we have:

Screenshot from 2024-07-24 10-18-38

How about something like:

Most of us use the ASF Slack workspace and the Arrow Rust Discord
server
for discussions.

There are specific channels for Arrow, DataFusion, and the DataFusion subprojects (Ballista, Comet, Python).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Makes sense! Will update tonight.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated the language and added all of the channels. think that is what you meant.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

happy to change again if I misunderstood.

Copy link
Member

@andygrove andygrove left a comment

Choose a reason for hiding this comment

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

Thanks @ajmarcus

@alamb
Copy link
Contributor

alamb commented Jul 25, 2024

🚀

@alamb alamb merged commit dcdcc25 into apache:main Jul 25, 2024
5 checks passed
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants