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

chore: update readme with new instructions for UI #595

Merged
merged 4 commits into from
Sep 29, 2021

Conversation

HarikrishnanBalagopal
Copy link
Contributor

Signed-off-by: Harikrishnan Balagopal [email protected]

@github-actions
Copy link

Thanks for making a pull request! 😃
One of the maintainers will review and advise on the next steps.

@github-actions github-actions bot added the chore Maintenance work to keep everything working correctly. label Sep 28, 2021
@codecov
Copy link

codecov bot commented Sep 28, 2021

Codecov Report

Merging #595 (eea3634) into main (1b42f19) will not change coverage.
The diff coverage is n/a.

❗ Current head eea3634 differs from pull request most recent head 687b9cb. Consider uploading reports for the commit 687b9cb to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##             main     #595   +/-   ##
=======================================
  Coverage   11.81%   11.81%           
=======================================
  Files          52       52           
  Lines        4324     4324           
=======================================
  Hits          511      511           
  Misses       3744     3744           
  Partials       69       69           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1b42f19...687b9cb. Read the comment docs.

README.md Outdated
* Run `docker run -p 8080:8080 -v $PWD:/workspace -v /var/run/docker.sock:/var/run/docker.sock -it quay.io/konveyor/move2kube-aio:latest`
* Access the UI in `http://localhost:8080/`.
```
docker run --rm -it -p 8080:8080 -v "${PWD}/data:/move2kube-api/data" -v /var/run/docker.sock:/var/run/docker.sock quay.io/konveyor/move2kube-ui:latest
Copy link
Member

Choose a reason for hiding this comment

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

Let's remove the docker.sock. its only for advanced usecases.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

README.md Outdated
```
Then go to http://localhost:8080 in a browser

More detailed instructions, including how to build the image from source, can be found here: https://github.com/konveyor/move2kube-ui#starting-the-ui
Copy link
Member

Choose a reason for hiding this comment

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

Make 'here' a link, instead of full url.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's good practise not to hide links. People like to see where they are being taken to.

@ashokponkumar ashokponkumar enabled auto-merge (squash) September 29, 2021 10:14
@ashokponkumar ashokponkumar merged commit b434743 into konveyor:main Sep 29, 2021
@HarikrishnanBalagopal HarikrishnanBalagopal linked an issue Sep 29, 2021 that may be closed by this pull request
@HarikrishnanBalagopal HarikrishnanBalagopal deleted the chore/updatereadme branch September 29, 2021 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Maintenance work to keep everything working correctly.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

UI image does not work with Podman
2 participants