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.

Sign up for GitHub

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

Jump to bottom

add Tests #3

Merged
merged 27 commits into from
Jul 30, 2023
Merged

add Tests #3

merged 27 commits into from
Jul 30, 2023

Conversation

0xbe7a
Copy link
Contributor

@0xbe7a 0xbe7a commented Jul 29, 2023

No description provided.

@0xbe7a 0xbe7a requested a review from pavelzw as a code owner July 29, 2023 19:57
pavelzw
pavelzw reviewed Jul 29, 2023
View reviewed changes
tests/test_parse_body.py Outdated Show resolved Hide resolved
pavelzw
pavelzw reviewed Jul 29, 2023
View reviewed changes
pixi.toml Outdated Show resolved Hide resolved
pavelzw
pavelzw approved these changes Jul 29, 2023
View reviewed changes
Copy link
Member

@pavelzw pavelzw left a comment

Choose a reason for hiding this comment

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

Awesome!

Do you think it makes sense to add some xfail tests for the stuff we haven't covered yet but will do?

tests/test_parse_body.py Outdated Show resolved Hide resolved
tests/test_parse_body.py Show resolved Hide resolved
polarify/main.py Outdated Show resolved Hide resolved
pavelzw
pavelzw reviewed Jul 29, 2023
View reviewed changes
README.md Outdated Show resolved Hide resolved
0xbe7a and others added 2 commits July 30, 2023 01:44
@0xbe7a @pavelzw
spelling
745a067 
Co-authored-by: Pavel Zwerschke <[email protected]>
@0xbe7a
Moved to #5
19ef4d2
@pavelzw pavelzw mentioned this pull request Jul 29, 2023
Open
@pavelzw pavelzw mentioned this pull request Jul 30, 2023
Merged
@0xbe7a 0xbe7a enabled auto-merge (squash) July 30, 2023 12:57
@0xbe7a 0xbe7a merged commit 1b9ce8e into main Jul 30, 2023
7 checks passed
@0xbe7a 0xbe7a deleted the tests branch July 30, 2023 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pavelzw pavelzw pavelzw approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@0xbe7a @pavelzw