Skip to content

Update README.md #884

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.

Sign up for GitHub

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

Jump to bottom
Merged
pirate merged 1 commit into from
Oct 28, 2021
Merged

Update README.md #884

pirate merged 1 commit into from
Oct 28, 2021

Conversation

@adamwolf
Copy link
Contributor

@adamwolf adamwolf commented Oct 28, 2021

Thanks, folks! Feel free to accept this if you find it helpful.

Summary

This PR fixes some dev instructions when setting up the Docker-y dev environment.

Before this, I would get something like

❯ docker run -it -p 8000:8000 \
    -v $PWD/data:/data \
    -v $PWD/archivebox:/app/archivebox \
    archivebox server 0.0.0.0:8000 --debug --reload

[i] [2021-10-28 14:54:17] ArchiveBox v0.6.3: archivebox server 0.0.0.0:8000 --debug --reload
    > /data

[X] No archivebox index found in the current directory.
    /data

    Hint: Are you running archivebox in the right folder?
        cd path/to/your/archive/folder
        archivebox [command]

    Hint: To create a new archive collection or import existing data in this folder, run:
        archivebox init

Related issues

Changes these areas

  • Bugfixes
  • Feature behavior
  • Command line interface
  • Configuration options
  • Internal architecture
  • Snapshot data layout on disk

Update README.md
3b6ef76 
Bind the data directory when initializing a dev environment using
Docker.
@pirate pirate merged commit c20c3be into ArchiveBox:dev Oct 28, 2021
@adamwolf
Copy link
Contributor Author

adamwolf commented Oct 31, 2021

Thanks! If you don't mind, could you add hacktoberfest-accepted as a label here too?

@pirate pirate added size: easy hacktoberfest-accepted status: done Work is completed and released (or scheduled to be released in the next version) touches: docs labels Nov 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

size: easy status: done Work is completed and released (or scheduled to be released in the next version) touches: docs

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@adamwolf @pirate