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

Fix D400 Issue #7776 #7829

Merged
merged 77 commits into from
Jun 14, 2023
Merged

Fix D400 Issue #7776 #7829

merged 77 commits into from
Jun 14, 2023

Conversation

Rondiz
Copy link
Contributor

@Rondiz Rondiz commented Jun 13, 2023

No description provided.

Rondiz added 30 commits June 13, 2023 12:49
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
D400: First line should end with a period
@Rondiz Rondiz marked this pull request as ready for review June 13, 2023 17:26
@Rondiz Rondiz marked this pull request as draft June 13, 2023 18:36
@Rondiz Rondiz marked this pull request as ready for review June 13, 2023 21:54
python/pyiceberg/catalog/__init__.py Outdated Show resolved Hide resolved
python/pyiceberg/conversions.py Show resolved Hide resolved
python/pyiceberg/io/pyarrow.py Outdated Show resolved Hide resolved
python/pyiceberg/io/pyarrow.py Outdated Show resolved Hide resolved
python/pyiceberg/serializers.py Outdated Show resolved Hide resolved
@Fokko Fokko merged commit 461f8d0 into apache:master Jun 14, 2023
7 checks passed
@Fokko
Copy link
Contributor

Fokko commented Jun 14, 2023

Thanks @Rondiz for the work, this is awesome! 🥳

@mderoy
Copy link
Contributor

mderoy commented Jun 14, 2023

adding a comment with #7776 so that it gets referenced in the ticket :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants