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

Update documentation README for gh-pages branch #619

Merged
merged 3 commits into from
Aug 23, 2022
Merged

Update documentation README for gh-pages branch #619

merged 3 commits into from
Aug 23, 2022

Conversation

taylorfturner
Copy link
Contributor

Bullet point out the exact steps for re-generating gh pages for Data Profiler (w/ context for exactly from where the commands should be run)

@taylorfturner taylorfturner added Bug Something isn't working Documentation Improvements or additions to documentation Low Priority Cosmetic improvement or minor bug labels Aug 22, 2022
@taylorfturner taylorfturner self-assigned this Aug 22, 2022
README.md Outdated
update the documentation (probably main).
1. Either with an existing clone of `capitalone/dataprofiler` or clone the `capitalone/dataprofiler` reposotory to your local computer with the following command:
```bash
git clone https://github.com/capitalone/DataProfiler dataprofiler
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we need to enforce downloading it with that name?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

don't have to

Copy link
Collaborator

Choose a reason for hiding this comment

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

Let's not enforce it. In the case someone sets up a second repo, this could cause conflicts.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

gotcha so will default to DataProfiler ... could be the same issue with conflicts, but most folks should have a capitalone/dataprofiler clone already so this clone command shouldn't need to happen for most folks on our team

README.md Show resolved Hide resolved
@JGSweets JGSweets enabled auto-merge (squash) August 23, 2022 14:43
@JGSweets JGSweets removed the Bug Something isn't working label Aug 23, 2022
Copy link
Collaborator

@JGSweets JGSweets left a comment

Choose a reason for hiding this comment

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

revert clone statement

README.md Outdated Show resolved Hide resolved
@JGSweets JGSweets merged commit 23a52b6 into capitalone:gh-pages Aug 23, 2022
@taylorfturner taylorfturner deleted the docs/README_generate_instructions branch October 4, 2022 14:42
JGSweets pushed a commit that referenced this pull request Oct 5, 2022
* update README steps

* update README directions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Improvements or additions to documentation Low Priority Cosmetic improvement or minor bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants