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

Dockerfile adjustments #175

Merged
merged 2 commits into from
Dec 6, 2022
Merged

Dockerfile adjustments #175

merged 2 commits into from
Dec 6, 2022

Conversation

paulosgf
Copy link
Collaborator

@paulosgf paulosgf commented Dec 5, 2022

Description

Add docker run argument to separate honeybot init and run fazes

How Has This Been Tested?

On local docker image

Additional context

  • Fix the unnamed image problem
  • Change the base image from python to ubuntu's latest

Add any other context or screenshots about the pull request here.

Checklist

  • I have read, understood and followed our Contributing Guide
  • My code follows the code style of this project
  • I have updated the documentation accordingly
  • I added my country flag to the README.md

Code of Conduct

By submitting this issue, you agree to follow our Code of Conduct

  • I agree to follow this project's Code of Conduct

Copy link
Member

@Abdur-rahmaanJ Abdur-rahmaanJ left a comment

Choose a reason for hiding this comment

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

@paulosgf Just add building and running section on the README

- Update README.md with new docker use info
@Abdur-rahmaanJ Abdur-rahmaanJ changed the title Dockerfile adjusts Dockerfile adjustments Dec 6, 2022
README.md Show resolved Hide resolved
@Abdur-rahmaanJ Abdur-rahmaanJ merged commit ff02ce3 into pyhoneybot:master Dec 6, 2022
@MrTanoshii MrTanoshii added documentation Improvements or additions to documentation ci-cd Continuous integration/continuous delivery labels Dec 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-cd Continuous integration/continuous delivery documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants