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

[MRG] Minor update to install instructions and license statements #558

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

DoomedJupiter
Copy link
Collaborator

Provides minor updates to the install instructions because installing ghostscript is no longer required to use the package. Also the [base] install option is not available in the .toml and generates a warning on install (still installs correctly though).

Adds an additional license statement to explicitly identify the second license file in the codebase, associated with the docs theme. Resolves #346

Addresses issue camelot-dev#346 by adding a statement identifying the second license for the docs theme
Addresses camelot-dev#346 by adding a line calling out the other license explicitly.
streamlines install instructions since ghostscript is no longer needed. Removes the [base] option since it does not exists in the .toml and generates a warning on install
@DoomedJupiter DoomedJupiter changed the title Minor update to install instructions and license statements [MRG] Minor update to install instructions and license statements Mar 12, 2025
@bosd
Copy link
Collaborator

bosd commented Mar 12, 2025

Thanks, I'm thinking it would be nice to also list the other install options.
Back at the previous repo, the package was designed to be light wheight so it could be run on headless servers.
By making the plotting librarys optional.

It seems that those options have been lost when we transferred repo's.
Will look into it later..

@bosd bosd added documentation Improvements or additions to documentation dependencies Pull requests that update a dependency file labels Mar 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OSS License compatibility question
2 participants