Skip to content
This repository was archived by the owner on Aug 28, 2025. It is now read-only.

Conversation

speediedan
Copy link
Contributor

Before submitting

  • Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
  • Did you make sure to update the docs?
  • Did you write any new necessary tests?

What does this PR do?

Fixes #162 .

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

@codecov
Copy link

codecov bot commented Apr 18, 2022

Codecov Report

Merging #163 (ee13b72) into main (eeb2a30) will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff         @@
##           main   #163   +/-   ##
===================================
  Coverage    75%    75%           
===================================
  Files         2      2           
  Lines       370    370           
===================================
  Hits        276    276           
  Misses       94     94           

@Borda Borda merged commit 22717e7 into Lightning-AI:main Apr 18, 2022
@speediedan speediedan deleted the makefile_update branch May 12, 2022 19:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Makefile reference to renamed assistant.py func

2 participants