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

Updated documentation #72

Merged
merged 5 commits into from
Nov 9, 2020
Merged

Updated documentation #72

merged 5 commits into from
Nov 9, 2020

Conversation

claesgill
Copy link
Contributor

Closes/fixes/resolves issue(s)?

Closes #60

What was added/changed/fixed?

  • Additional information about proxies and Presto CLI

Related issue(s)? [Optional]

Others [Optional]

Checklist (after created PR)

  • Added reviewers
  • Added assignee(s)
  • Added label(s)
  • Added to project
  • Added to milestone

@claesgill claesgill added this to the 0.3.0 milestone Nov 6, 2020
@claesgill claesgill added this to Review in progress in Team DataStack via automation Nov 6, 2020
@claesgill claesgill self-assigned this Nov 6, 2020
@claesgill claesgill mentioned this pull request Nov 6, 2020
3 tasks
pdmthorsrud
pdmthorsrud previously approved these changes Nov 6, 2020
README.md Outdated Show resolved Hide resolved
Co-authored-by: Nikita Zhevnitskiy <nikita.zhevnitskiy@gmail.com>
README.md Outdated Show resolved Hide resolved
Co-authored-by: dangernil <71001093+dangernil@users.noreply.github.com>
pdmthorsrud
pdmthorsrud previously approved these changes Nov 9, 2020
@pdmthorsrud
Copy link
Contributor

Should we add the change of retries to the changelog or nah? I guess that is something that "should" have been there to begin with, so let's call it a hotfix?

@claesgill
Copy link
Contributor Author

Should we add the change of retries to the changelog or nah? I guess that is something that "should" have been there to begin with, so let's call it a hotfix?

I think it's part of issue #52 which is linked in the changelog.

Team DataStack automation moved this from Review in progress to Reviewer approved Nov 9, 2020
@claesgill claesgill merged commit 11ef41e into master Nov 9, 2020
Team DataStack automation moved this from Reviewer approved to Pull Requests merged and closed Nov 9, 2020
@claesgill claesgill deleted the issue_60_additional_info_readme branch November 9, 2020 09:33
@fredrikhgrelland fredrikhgrelland removed this from Pull Requests merged and closed in Team DataStack Mar 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Additional information in README.md
4 participants