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

Upload logs as artifact #4640

Merged

Conversation

waynehamadi
Copy link
Contributor

@waynehamadi waynehamadi commented Jun 10, 2023

Background

Upload logs as artifacts so we can see why some tests failed.

Changes

  • added the github action

Documentation

Test Plan

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.
  • I have run the following commands against my code to ensure it passes our linters:
    black .
    isort .
    mypy
    autoflake --remove-all-unused-imports --recursive --ignore-init-module-imports autogpt tests --in-place

@vercel
Copy link

vercel bot commented Jun 10, 2023

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

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Ignored (Inspect) Jun 10, 2023 10:03pm

@codecov
Copy link

codecov bot commented Jun 10, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.05 ⚠️

Comparison is base (6b9e3b2) 68.86% compared to head (bd72aeb) 68.81%.

❗ Current head bd72aeb differs from pull request most recent head ba7bcf8. Consider uploading reports for the commit ba7bcf8 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4640      +/-   ##
==========================================
- Coverage   68.86%   68.81%   -0.05%     
==========================================
  Files          72       72              
  Lines        3597     3585      -12     
  Branches      578      574       -4     
==========================================
- Hits         2477     2467      -10     
+ Misses        928      927       -1     
+ Partials      192      191       -1     

see 3 files with indirect coverage changes

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

@waynehamadi waynehamadi merged commit c1ee8cb into Significant-Gravitas:master Jun 10, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants