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

Docker Image Addition #49

Merged
merged 7 commits into from
Jan 25, 2024
Merged

Docker Image Addition #49

merged 7 commits into from
Jan 25, 2024

Conversation

kauevestena
Copy link
Contributor

Describe your changes and approach used

Well, I saw many issues on dependencies, that are a common shared hell, and thank you, universe, there's Docker.

Fixes #39 #38 #30 #26 #20 #13 and probably others. Well at least, provided that user got Docker Engine Installed =D

Checklist before requesting a review

  • I have performed a self-review of my code.
  • I have performed linting on my code.
  • I have linked the related issue.
  • The code is documented accordingly.
  • If it is a core feature, I have added thorough tests.

Also,
@luca-medeiros by your name I will ask: Tu é brasileiro?

Kindest regards,
Kauê

Copy link
Owner

@luca-medeiros luca-medeiros left a comment

Choose a reason for hiding this comment

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

@kauevestena Hey, really appreciate you taking the time to solve this recurring issue. Bet lots of people will use it well.

Wrote a few suggestions that we should fix before merging!
Sim, sou brasileiro :)

Dockerfile Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@kauevestena
Copy link
Contributor Author

I think that's it for now!

And as we say: é nós que avua meu caro!

Copy link
Owner

@luca-medeiros luca-medeiros left a comment

Choose a reason for hiding this comment

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

LGTM! valews

@luca-medeiros luca-medeiros merged commit f4ca296 into luca-medeiros:main Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AttributeError: module 'lightning' has no attribute 'LightningApp'
2 participants