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

Add comment about password for the ISPRS Potsdam dataset #1627

Merged
merged 1 commit into from
Dec 21, 2022

Conversation

ameier3
Copy link
Contributor

@ameier3 ameier3 commented Dec 20, 2022

As mentioned by @AdeelH here I added a comment on where to find the password in the master branch

Overview

Brief description of what this PR does, and why it is needed.

Checklist

  • Added needs-backport label if PR is bug fix that applies to previous minor release
  • Ran scripts/format_code and committed any changes
  • [x ] Documentation updated if needed
  • PR has a name that won't get you publicly shamed for vagueness

Notes

Optional. Ancillary topics, caveats, alternative strategies that didn't work out, anything else.

Testing Instructions

  • How to test this PR
  • Prefer bulleted description
  • Start after checking out this branch
  • Include any setup required, such as rebuilding the Docker image.
  • Include test case, and expected output if not captured by automated tests.

Closes #XXX

As mentioned by @AdeelH  [here](azavea#1597 (comment)) I added a comment on where to find the password in the master branch
Copy link
Collaborator

@AdeelH AdeelH 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!

@AdeelH AdeelH changed the title Add comment about password Add comment about password for the ISPRS Potsdam dataset Dec 21, 2022
@AdeelH AdeelH merged commit a9b46b7 into azavea:master Dec 21, 2022
@AdeelH AdeelH added the needs-backport This PR needs to be backported to release branches label Dec 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-backport This PR needs to be backported to release branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants