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

Fix broken hyperlink in the documentation #972

Open
rakeshgowerneni opened this issue Jul 27, 2023 · 14 comments
Open

Fix broken hyperlink in the documentation #972

rakeshgowerneni opened this issue Jul 27, 2023 · 14 comments
Assignees
Labels
Documentation Improvements or additions to documentation good_first_issue Label for a more simple / introductory issue Help Wanted Extra attention is needed

Comments

@rakeshgowerneni
Copy link

Please provide the issue you face regarding the documentation
The hyperlink in the below sentence in the Load a File section should change from

If the file type is not automatically identified (rare), you can specify them specifically, see section Specifying a Filetype or Delimiter.

to

If the file type is not automatically identified (rare), you can specify them specifically, see section Specifying a Filetype or Delimiter.

I am trying to fix the documentation as a way to create my first PR on the repo.

@rakeshgowerneni rakeshgowerneni added the Documentation Improvements or additions to documentation label Jul 27, 2023
@JGSweets
Copy link
Collaborator

@rakeshgowerneni that's a good catch! I'm curious why sphinx did not generate that properly. Thoughts on how to fix? We welcome PRs and would love to see you contribute!

@taylorfturner
Copy link
Contributor

@rakeshgowerneni -- Check out our contribution guide here for creating pull requests.

@taylorfturner
Copy link
Contributor

When you implement the fix @rakeshgowerneni, be sure its not specifying a specific version in the hyperlink. Rather, it should be agnostic to the version and just resolve to the proper location within whatever docs are live at that time.

If the file type is not automatically identified (rare), you can specify them specifically, see section [Specifying a Filetype or Delimiter](https://capitalone.github.io/DataProfiler/docs/0.10.1/html/profiler.html?highlight=filetype#specifying-a-filetype-or-delimiter).

@taylorfturner taylorfturner added Help Wanted Extra attention is needed good_first_issue Label for a more simple / introductory issue labels Sep 12, 2023
@thebadcoder96
Copy link

Hey is anyone working on this? Can I try this out?

@taylorfturner
Copy link
Contributor

taylorfturner commented Jan 6, 2024 via email

@taylorfturner
Copy link
Contributor

@thebadcoder96 let me know if you have anymore questions -- the team is back at it today. Thanks for your interest!

@thebadcoder96
Copy link

Thank you for the response! I will go through the readme file that you mentioned and let you know if I have any questions.

@taylorfturner
Copy link
Contributor

Thank you for the response! I will go through the readme file that you mentioned and let you know if I have any questions.

Thanks @thebadcoder96!

@thebadcoder96
Copy link

Here is how I think we can solve the issue. I do not have much idea on sphinx but I am looking into that as well.

  • We can run a Python script that prints out the current/live version of Data Profiler.
  • Trigger a custom GitHub Actions workflow when there is a push to main and use the Python script to update the readme file using the latest version. Probably use sed for updating the readme file.

Wdyt? Any other thoughts are welcome since I do not have full context on how everything works within the Data Profiler, I might be wrong as well.

@taylorfturner
Copy link
Contributor

taylorfturner commented Jan 8, 2024

I think we can actually re-use the code written out here.

That script will have:

  • the proper version tag
  • we can simply find the spot in the pertinent file to ensure its always referencing the most up-to-date build of the docs.

That should work! For updating docs, all changes will happen on your fork's branch of dev-gh-pages and you will make the changes in docs/update_documentation.py or docs/src. Once you get that point, though, just ping me and I'll be glad to answer any questions (or feel free to add to README for gh-pages branches for improved direction clarity)

@thebadcoder96
Copy link

Please correct me if I am wrong, but the issue mentioned here pertains to the readme file of the Data Profiler Github. In the documentation website there is no hyperlink and the sentence is also a bit different.

From my understanding, docs/update_documentation.py updates the GitHub pages documentation, which is NOT the one we want to update right?

@taylorfturner
Copy link
Contributor

@thebadcoder96 yes, you are correct. I was confusing something else.

In this case I would key off of the version.py file and try to reference that in the README.md as a consistent reference point for the proper version for the hyperlink... there might be another way too to reference the index.html and tack on a #specifying-a-hyoerlink(or something like that) to the hyperlink in the README

@taylorfturner
Copy link
Contributor

@thebadcoder96 the URL should be

https://capitalone.github.io/DataProfiler/docs/<current_version_tag>/html/profiler.html?highlight=filetype#specifying-a-filetype-or-delimiter

We'll want the version tag to be a reference and not a manual change every time we make a new release of dataprofiler.

@thebadcoder96
Copy link

Since you mentioned version tag, I think we can use git describe --tags to get the current version tag and then update the readme file.

I have created a PR with the proposed changes, please review #1080

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 good_first_issue Label for a more simple / introductory issue Help Wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

6 participants