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

switching from unittest to pytest in test_json_parser #3481

Merged
merged 6 commits into from
May 18, 2023

Conversation

rihp
Copy link
Contributor

@rihp rihp commented Apr 28, 2023

Background

we are deprecating unittest in favor of pytest, this continues with the work in

Changes

only to the file test_json_parser

Documentation

not necesary

Test Plan

run pytest

PR Quality Checklist

  • My pull request is atomic and focuses on a single change.
  • I have thoroughly tested my changes with multiple different prompts.
  • I have considered potential risks and mitigations for my changes.
  • I have documented my changes clearly and comprehensively.
  • I have not snuck in any "extra" small tweaks changes

@vercel
Copy link

vercel bot commented Apr 28, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 18, 2023 5:54am

@github-actions github-actions bot added size/xl and removed size/l labels Apr 28, 2023
@github-actions
Copy link
Contributor

This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size

@codecov
Copy link

codecov bot commented Apr 28, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (dc41db9) 62.67% compared to head (5a01f66) 62.67%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3481   +/-   ##
=======================================
  Coverage   62.67%   62.67%           
=======================================
  Files          74       74           
  Lines        3400     3400           
  Branches      495      495           
=======================================
  Hits         2131     2131           
  Misses       1120     1120           
  Partials      149      149           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@rihp rihp marked this pull request as ready for review April 28, 2023 17:12
@Boostrix Boostrix mentioned this pull request May 3, 2023
7 tasks
@p-i-
Copy link
Contributor

p-i- commented May 5, 2023

This is a mass message from the AutoGPT core team.
Our apologies for the ongoing delay in processing PRs.
This is because we are re-architecting the AutoGPT core!

For more details (and for infor on joining our Discord), please refer to:
https://github.com/Significant-Gravitas/Auto-GPT/wiki/Architecting

@ntindle ntindle self-assigned this May 16, 2023
@vercel
Copy link

vercel bot commented May 16, 2023

Deployment failed with the following error:

Resource is limited - try again in 57 minutes (more than 100, code: "api-deployments-free-per-day").

@vercel vercel bot temporarily deployed to Preview May 18, 2023 05:54 Inactive
@ntindle ntindle merged commit c4b32e0 into Significant-Gravitas:master May 18, 2023
ppetermann pushed a commit to ppetermann/Auto-GPT that referenced this pull request May 22, 2023
@ntindle ntindle added this to the v0.4.0 Release milestone May 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants