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(artifacts): correct parameter name in docstring example #4528

Merged
merged 3 commits into from Nov 28, 2022
Merged

docs(artifacts): correct parameter name in docstring example #4528

merged 3 commits into from Nov 28, 2022

Conversation

ngrayluna
Copy link
Contributor

docs(artifacts): corrected parameter name in docstring example for .add_dir method. The correct paramater name is name, not path.

Fixes: #4443

Description

What does the PR do?
Corrects parameter name in docstring example for .add_dir method. The correct paramater name is name, not path.

Testing

How was this PR tested?

Checklist

  • [ X] Include reference to internal ticket "Fixes WB-NNNN" and/or GitHub issue "Fixes #NNNN" (if applicable)
  • [ X] Ensure PR title compliance with the conventional commits standards

Copy link
Contributor

@moredatarequired moredatarequired left a comment

Choose a reason for hiding this comment

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

Thank you!

@moredatarequired moredatarequired changed the title docs(artifacts): corrected parameter name in docstring example. docs(artifacts): correct parameter name in docstring example Nov 23, 2022
@github-actions github-actions bot added cc-docs and removed cc-docs labels Nov 23, 2022
@dmitryduev dmitryduev merged commit cb77d21 into wandb:main Nov 28, 2022
jameshennessytempus added a commit to jameshennessytempus/wandb that referenced this pull request Nov 29, 2022
docs(artifacts): correct parameter name in docstring example (wandb#4528)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect parameter name in docstring for artifact.add_dir()
3 participants