-
Notifications
You must be signed in to change notification settings - Fork 60.6k
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
Update package-immutability.md #190
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fixes github#122 for https://help.github.com/packages/publishing-and-managing-packages/deleting-a-package#reserved-package-versions-and-names and every other page that depends on this reusable.
Thanks @casals 💝 I'll send this over to the @github/docs-content-ecosystem team for review! |
janiceilene
added
ecosystem
This issue or pull request should be reviewed by the Docs Ecosystem team
packages
This issue or pull request should be reviewed by the docs packages team
labels
Oct 7, 2020
skedwards88
reviewed
Oct 9, 2020
Applying requested changes
skedwards88
approved these changes
Oct 9, 2020
jnidzwetzki
pushed a commit
to jnidzwetzki/docs
that referenced
this pull request
Oct 6, 2022
* NFL tutorial draft * Wording * Remove redundant paragraph * Add comments * Wording * Add indexes * Add to query section (github#190) * Add to query section * make changes, delete python to put focus on sql * remove numpy Co-authored-by: Miranda Auhl <miranda@timescale.com> * Minor changes * Varchar --> text * Small changes from review Co-authored-by: Lana Brindley <github@lanabrindley.com> * Small changes part 2 Co-authored-by: Lana Brindley <github@lanabrindley.com> * Wording * Add player position and movement visualization * Take out python prerequisites * add other queries and fill analytics section (github#193) * add other queries and fill analytics section * updates * one last change * one last change to help merge Co-authored-by: Miranda Auhl <miranda@timescale.com> * add imag urls * Additional info about tables, fields * Add download links for datasets * update images to correct folder location * Add play drawing code and images * Fix image links * Add create_hypertable * Formatting and links * reorganization of the NFL tutorial, breaking it up into multiple pages. (github#202) * reorganization of the NFL tutorial, breaking it up into multiple pages. * Fixing links on first page * Updating both index pages with links and text * Apply suggestions from code review Lots of good cleanup thanks to Miranda and Lana! Co-authored-by: Lana Brindley <github@lanabrindley.com> Co-authored-by: mirandaauhl <82287545+mirandaauhl@users.noreply.github.com> * Remove "TimescaleDB Toolkit" in place of hyperfunction * Made the tutorial more generic about time-series data * cleaned up section names in left nav * Added a little more exposition * Apply suggestions from code review add query info in relational page * Add queries to relational page * Apply suggestions from code review add comma to fix query * fix one part of the queries * Apply suggestions from code review apply cagg comments * Apply suggestions from code review update titles * add tables and some exposition Co-authored-by: Ryan Booz <ryan@timescale.com> Co-authored-by: mirandaauhl <82287545+mirandaauhl@users.noreply.github.com> Co-authored-by: Miranda Auhl <miranda@timescale.com> Co-authored-by: Lana Brindley <github@lanabrindley.com> Co-authored-by: Prashant Sridharan <prashant_sridharan@hotmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why:
Fixes #122 .
What's being changed:
The original issue refers to this page only. However, the indicated text comes from the reusable being modified by this PR. New content for private packages as per the original issue author's suggestion.
Additional information
This PR affects not only the page referenced by the issue, but also every other page that depends on this reusable.
Check off the following: