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

Remove references to Spotify FOSS Slack #5149

Merged
merged 2 commits into from
Jan 10, 2024

Conversation

BalestraPatrick
Copy link
Member

The FOSS Slack is being shut down. We will continue use GitHub issues, discussions, etc. for receiving feedback and bug reports.

Copy link

codecov bot commented Jan 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (164f963) 63.44% compared to head (23736a3) 63.43%.

❗ Current head 23736a3 differs from pull request most recent head 25e85aa. Consider uploading reports for the commit 25e85aa to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5149      +/-   ##
==========================================
- Coverage   63.44%   63.43%   -0.01%     
==========================================
  Files         291      291              
  Lines       10836    10836              
  Branches      744      744              
==========================================
- Hits         6875     6874       -1     
- Misses       3961     3962       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@@ -5,7 +5,7 @@ Scio is developed and maintained by an infrastructure team at Spotify. It is the

# Submitting issues

Feel free to discuss issues in the #scio channel on Spotify FOSS Slack (get an invite [here](https://slackin.spotify.com/)) or [Google Group](https://groups.google.com/forum/#!forum/scio-users) first.
Feel free to discuss issues in the [Google Group](https://groups.google.com/forum/#!forum/scio-users) first.
Copy link
Contributor

Choose a reason for hiding this comment

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

We've just enabled GitHub discussions on scio for that purpose

@@ -14,7 +14,6 @@ Example Scio pipelines and tests can be found under @github[scio-examples](/scio
See @scaladoc[Scio Scaladocs](com.spotify.scio.index) for current API documentation.

## Getting help
- `#scio` channel on Spotify FOSS Slack, get an invite [here](https://slackin.spotify.com/)
Copy link
Contributor

Choose a reason for hiding this comment

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

Replace with GitHub discussion

Copy link
Member Author

Choose a reason for hiding this comment

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

Added now

@RustedBones
Copy link
Contributor

RustedBones commented Jan 10, 2024

This is expected that CI step Generate Site fails for PRs from fork. We'll fix that soon

@RustedBones RustedBones merged commit 39be44f into spotify:main Jan 10, 2024
8 of 9 checks passed
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.

3 participants