Skip to content

Removed the --pre flag for the install command. #948

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
May 5, 2023
Merged

Removed the --pre flag for the install command. #948

merged 2 commits into from
May 5, 2023

Conversation

@kevinlacaille
Copy link
Contributor

@kevinlacaille kevinlacaille commented May 5, 2023 •
edited
Loading

Related Issue(s):

Closes #947

Proposed Changes:

For inclusion in changelog (if applicable):

  1. Updated pip install command in the docs.

Not intended for changelog:

Diff of User Interface

Old behavior:
$ pip install planet --pre --user
New behavior:
$ pip install planet

PR Checklist:

  • This PR is as small and focused as possible
  • If this PR includes proposed changes for inclusion in the changelog, the title of this PR summarizes those changes and is ready for inclusion in the Changelog.
  • I have updated docstrings for function changes and docs in the 'docs' folder for user interface / behavior changes
  • [] This PR does not break any examples or I have updated them

(Optional) @mentions for Notifications:

@kevinlacaille kevinlacaille added this to the 2.0.x: Bug fixes and Docs milestone May 5, 2023
@kevinlacaille kevinlacaille self-assigned this May 5, 2023
@kevinlacaille kevinlacaille marked this pull request as ready for review May 5, 2023 19:15
@cholmes cholmes merged commit e808ff0 into main May 5, 2023
@kevinlacaille kevinlacaille deleted the update-pip-install-docs-947 branch May 5, 2023 19:54
JuLeeAtPlanet
JuLeeAtPlanet reviewed May 5, 2023
View reviewed changes
Copy link
Contributor

@JuLeeAtPlanet JuLeeAtPlanet left a comment

Choose a reason for hiding this comment

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

lgtm

@jreiberkyle jreiberkyle mentioned this pull request May 11, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cholmes cholmes cholmes approved these changes

+1 more reviewer

@JuLeeAtPlanet JuLeeAtPlanet JuLeeAtPlanet left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@kevinlacaille kevinlacaille

Labels

documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

update pip install command in docs

4 participants

@kevinlacaille @cholmes @JuLeeAtPlanet