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

Update package-immutability.md #190

Merged
merged 4 commits into from
Oct 9, 2020
Merged

Update package-immutability.md #190

merged 4 commits into from
Oct 9, 2020

Conversation

casals
Copy link
Contributor

@casals casals commented Oct 7, 2020

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:

@janiceilene
Copy link
Contributor

Thanks @casals 💝 I'll send this over to the @github/docs-content-ecosystem team for review!

@janiceilene 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 skedwards88 merged commit 4dd9e02 into github:main 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
Labels
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update GitHub Packages reserved names and versions section
3 participants