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 guide_dataset.rst #4162

Merged
merged 1 commit into from
Apr 16, 2024
Merged

Update guide_dataset.rst #4162

merged 1 commit into from
Apr 16, 2024

Conversation

varunity
Copy link
Contributor

Correcting a typo. No tests or extra documentation needed.

fixes [org/repo/issue#]

  • Test coverage exists
  • Documentation exists

QA Steps

  • Read the edited sentence, see if it makes more sense.

Correcting a typo
@dafeder
Copy link
Member

dafeder commented Apr 16, 2024

Hi @varunity! 👋🏽 I think "distribution" is technically correct here but I agree "file" is too, and is clearer. Thanks for the PR, will merge once we have this one flaky test passing again.

@dafeder dafeder merged commit 24bd853 into GetDKAN:2.x Apr 16, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants