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: add contributor notes on where to find documentation branches #1113

Merged
merged 5 commits into from
Apr 8, 2024

Conversation

carlsonp
Copy link
Contributor

No description provided.

@carlsonp carlsonp requested a review from a team as a code owner March 12, 2024 18:38
Copy link
Contributor

@taylorfturner taylorfturner left a comment

Choose a reason for hiding this comment

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

open into dev branch instead of main but otherwise LGTM

@carlsonp carlsonp changed the base branch from main to dev March 12, 2024 18:44
@taylorfturner taylorfturner enabled auto-merge (squash) March 12, 2024 18:46
@taylorfturner taylorfturner added the Documentation Improvements or additions to documentation label Mar 12, 2024
@@ -106,3 +106,6 @@ the documentation.

## Updating Dependencies
If you make changes to the `requirements` text files, please also update the `additional_dependencies` list under the `mypy` hook in `.pre-commit-config.yaml`. This is necessary for accurate type-checking.

## Contributing Documentation Changes and Fixes
Please see the `gh-pages` and `dev-gh-pages` branches for all documentation.
Copy link
Contributor

@taylorfturner taylorfturner Mar 12, 2024

Choose a reason for hiding this comment

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

Technically a PR for updating the documentation should only go into dev-gh-pages -- and when we do a release a PR will be opened to gh-pages. Follow-up clarity worthwhile for this new section

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good idea on clarification, see 5e36e17

Copy link
Contributor

Choose a reason for hiding this comment

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

looking good @carlsonp

taylorfturner
taylorfturner previously approved these changes Mar 12, 2024
Copy link
Contributor

@taylorfturner taylorfturner left a comment

Choose a reason for hiding this comment

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

LGTM -- small comment for follow-up PR. Thanks for the contribution @carlsonp

auto-merge was automatically disabled March 12, 2024 19:17

Head branch was pushed to by a user without write access

Copy link
Contributor

@taylorfturner taylorfturner left a comment

Choose a reason for hiding this comment

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

slightly nuance clarification

.github/CONTRIBUTING.md Outdated Show resolved Hide resolved
@taylorfturner
Copy link
Contributor

Seeing a test error: E ValueError: Must install dask-expr to activate query planning probably related to E ModuleNotFoundError: No module named 'dask_expr'. Not seeing this on other PRs at the moment, but dask did make a release yesterday https://pypi.org/project/dask/#history

@taylorfturner
Copy link
Contributor

I approve this change, @carlsonp -- we'll just need this dask issue resolved so tests pass on the checks. If you have time, welcome to research and propose a fix. Otherwise, I'll have to see if I can get to it later; but this change itself is good! Thanks for the contributions @carlsonp!

@taylorfturner
Copy link
Contributor

@carlsonp you should be good to rebase onto dev now

@carlsonp
Copy link
Contributor Author

Rebased

@taylorfturner taylorfturner enabled auto-merge (squash) March 25, 2024 12:35
Copy link
Contributor

@taylorfturner taylorfturner left a comment

Choose a reason for hiding this comment

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

slight update @carlsonp

.github/CONTRIBUTING.md Outdated Show resolved Hide resolved
auto-merge was automatically disabled March 27, 2024 22:35

Head branch was pushed to by a user without write access

Co-authored-by: Taylor Turner <[email protected]>
Copy link
Contributor

@taylorfturner taylorfturner left a comment

Choose a reason for hiding this comment

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

LGTM

@taylorfturner
Copy link
Contributor

@carlsonp needs update branch now after #1114 merged

@taylorfturner taylorfturner merged commit eb45208 into capitalone:dev Apr 8, 2024
5 checks passed
abajpai15 pushed a commit to abajpai15/DataProfiler that referenced this pull request Apr 15, 2024
…apitalone#1113)

* docs: add contributor notes on where to find documentation branches

* docs: update documentation wording to spell out why `dev-gh-pages` and `gh-pages` branches exist for staging content

* docs: add note on fork

Co-authored-by: Taylor Turner <[email protected]>

* Update .github/CONTRIBUTING.md

Co-authored-by: Taylor Turner <[email protected]>

---------

Co-authored-by: Taylor Turner <[email protected]>
micdavis pushed a commit that referenced this pull request May 20, 2024
* Replace snappy with cramjam (#1091)

* add downloads tile (#1085)

* Replace snappy with cramjam

* Delete test_no_snappy

---------

Co-authored-by: Taylor Turner <[email protected]>

* Quick fix for dependency max pins (#1120)

* Fix dask_expr

* Keras and Tensorflow version fix

* Keras and Tensorflow version fix

* Fix keras bug

* pre-commit fix (#1122)

* docs: update test link to latest version (#1114)

* docs: add contributor notes on where to find documentation branches (#1113)

* docs: add contributor notes on where to find documentation branches

* docs: update documentation wording to spell out why `dev-gh-pages` and `gh-pages` branches exist for staging content

* docs: add note on fork

Co-authored-by: Taylor Turner <[email protected]>

* Update .github/CONTRIBUTING.md

Co-authored-by: Taylor Turner <[email protected]>

---------

Co-authored-by: Taylor Turner <[email protected]>

* update black version (#1131)

* Add memray max version (#1132)

* Bug fix for float precision calculation using categorical data with trailing zeros. (#1125)

* Revert "Bug fix for float precision calculation using categorical data with t…" (#1133)

This reverts commit d3159bd.

* fix

* make up to date

* yep, shouldn't change

* bump version

---------

Co-authored-by: Gábor Lipták <[email protected]>
Co-authored-by: abajpai15 <[email protected]>
Co-authored-by: Patrick Carlson <[email protected]>
Co-authored-by: James Schadt <[email protected]>
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.

4 participants