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

[DOCS] Fix relative links on github #4479

Merged
merged 10 commits into from
Mar 24, 2022

Conversation

andyjessen
Copy link
Contributor

This commit fixes relative links returning '404' error when accessed on https://github.com/great-expectations/great_expectations.

Definition of Done

Please delete options that are not relevant.

Fix relative links on github

This commit fixes relative links returning '404' error when accessed on https://github.com/great-expectations/great_expectations.
@netlify
Copy link

netlify bot commented Mar 20, 2022

👷 Deploy request for niobium-lead-7998 pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 1b7fca5

Copy link
Contributor

@austiezr austiezr left a comment

Choose a reason for hiding this comment

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

Awesome to see this come in; thank you so much @andyjessen !

I've noted out places where the updated links are broken; otherwise love to see this. 🚀

docs/intro.md Outdated Show resolved Hide resolved
docs/contributing/contributing_checklist.md Outdated Show resolved Hide resolved
Comment on lines 100 to 105
For more advanced configurations of Checkpoints, please see [How to configure a new Checkpoint using test_yaml_config](../../../guides/validation/checkpoints/how_to_configure_a_new_checkpoint_using_test_yaml_config.md).

### Additional Resources

* [How to configure a new Checkpoint using test_yaml_config](/docs/guides/validation/checkpoints/how_to_configure_a_new_checkpoint_using_test_yaml_config)
* [How to add validations data or suites to a Checkpoint](/docs/guides/validation/checkpoints/how_to_add_validations_data_or_suites_to_a_checkpoint)
* [How to configure a new Checkpoint using test_yaml_config](../../../guides/validation/checkpoints/how_to_configure_a_new_checkpoint_using_test_yaml_config.md)
* [How to add validations data or suites to a Checkpoint](../../../guides/validation/checkpoints/how_to_add_validations_data_or_suites_to_a_checkpoint.md)
Copy link
Contributor

Choose a reason for hiding this comment

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

../../guides/<path>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I couldn't get suggested path to work.

Copy link
Contributor

Choose a reason for hiding this comment

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

My bad; looks like these should be ./<file_name>

docs/reference/data_discovery.md Outdated Show resolved Hide resolved
docs/reference/data_discovery.md Outdated Show resolved Hide resolved
docs/reference/validation.md Outdated Show resolved Hide resolved
@austiezr austiezr added community devrel This item is being addressed by the Developer Relations Team labels Mar 21, 2022
Copy link
Contributor

@austiezr austiezr left a comment

Choose a reason for hiding this comment

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

I believe these are the last pieces; thank you for your effort on this! 🙇

Comment on lines 327 to 328
1. [Running a Checkpoint in Airflow using a `bash operator`](./how_to_use_great_expectations_with_airflow#option-1-running-a-checkpoint-with-a-bashoperator.md)
2. [Running a Checkpoint in Airflow using a `python operator`](./how_to_use_great_expectations_with_airflow#option-2-running-the-checkpoint-script-output-with-a-pythonoperator.md)
Copy link
Contributor

Choose a reason for hiding this comment

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

./<path>.md#<heading>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I fixed link in new commit.

@@ -451,14 +451,14 @@ You've successfully migrated your Great Expectations configuration to Cloud Comp

There are many ways to iterate and improve this initial version, which used a `bash operator` for simplicity. For information on more sophisticated ways of triggering Checkpoints, building our DAGs, and dividing our Data Assets into Batches using DataConnectors, please refer to the following documentation:

- [How to run a Checkpoint in Airflow using a `python operator`](/docs/deployment_patterns/how_to_use_great_expectations_with_airflow#option-2-running-the-checkpoint-script-output-with-a-pythonoperator).
- [How to run a Checkpoint in Airflow using a `python operator`](./how_to_use_great_expectations_with_airflow#option-2-running-the-checkpoint-script-output-with-a-pythonoperator.md).
Copy link
Contributor

Choose a reason for hiding this comment

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

./<path>.md#<heading>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I fixed link in new commit.

Copy link
Contributor

Choose a reason for hiding this comment

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

Hey @andyjessen ! I'm not seeing the latest commit re: above.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks @austiezr for catching my mistake. (I made commit on separate branch that got pruned.) I added to current branch and pushed.

@andyjessen
Copy link
Contributor Author

I believe these are the last pieces; thank you for your effort on this! 🙇

Hi @austiezr - I fixed links. I noticed another broken path in same file and fixed it as well.

Copy link
Contributor

@austiezr austiezr left a comment

Choose a reason for hiding this comment

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

Killer, thank you! 🙇

@austiezr austiezr enabled auto-merge (squash) March 24, 2022 13:47
@austiezr austiezr merged commit 033ae63 into great-expectations:develop Mar 24, 2022
@fjork3 fjork3 changed the title Fix relative links on github [DOCS] Fix relative links on github Mar 24, 2022
@andyjessen andyjessen deleted the fix-github-links branch March 25, 2022 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community devrel This item is being addressed by the Developer Relations Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants