Skip to content

docs: update MOLE.jl installation instructions - #481

Open
PandaHUN777 wants to merge 3 commits into
csrc-sdsu:mainfrom
PandaHUN777:docs/issue-443-julia-install
Open

PandaHUN777 wants to merge 3 commits into
csrc-sdsu:mainfrom
PandaHUN777:docs/issue-443-julia-install

Conversation

@PandaHUN777

Copy link
Copy Markdown

What type of PR is this?

  • Documentation

Description

Update the MOLE.jl installation instructions to use the Julia package manager, replacing the outdated note that the package was unavailable there.

Related issue

Closes #443

QA Instructions

Verified the package name, UUID, and repository subdirectory against the Julia General registry and the package metadata. git diff --check passes.

Added/updated tests?

  • Yes
  • No, documentation-only change.
  • I need help with writing tests

Read Contributing Guide and Code of Conduct

  • I have read the MOLE Contributing Guide.
  • I have read the MOLE Code of Conduct.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Related documentation still contains contradictory outdated installation instructions.

Review effort: Lite
Findings: 1 Medium severity

Open (1)
What changed in this PR

Updates MOLE.jl installation instructions to use Julia’s package manager.

Changes:

  • Replaces the outdated local-clone guidance with Pkg.add("MOLE").
File Summary
julia/​MOLE.jl/​README.md Updates package installation guidance.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread julia/MOLE.jl/README.md Outdated
## Installation

MOLE.jl is not yet available in the Julia's package manager. For now, this repository needs to be cloned locally in order to use the library in Julia.
MOLE.jl is available from the Julia package manager. In Julia, run `using Pkg; Pkg.add("MOLE")` to install it.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Updated docs/src/index.md to match the README's Julia package-manager installation instructions and removed the contradictory local-clone guidance. Documentation validation and git diff --check pass.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Updated both julia/MOLE.jl/README.md and docs/src/index.md to establish standard package-manager usage (using MOLE) without requiring a repository clone, and relocated the clone-based --project=. instructions into a dedicated 'Development and working from source' section. Documentation validation and git diff --check pass.

@codecov

codecov Bot commented Sep 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@PandaHUN777
PandaHUN777 force-pushed the docs/issue-443-julia-install branch from fa03022 to 4c6b581 Compare September 28, 2026 17:30
Comment thread julia/MOLE.jl/README.md Outdated
## Installation

MOLE.jl is not yet available in the Julia's package manager. For now, this repository needs to be cloned locally in order to use the library in Julia.
MOLE.jl is available from the Julia package manager. In Julia, run `using Pkg; Pkg.add("MOLE")` to install it.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you for your contribution!

This line reads well now. Could you please also add another sentence for the other pkg mode? That is ] add MOLE. Thank you

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Added the alternative Julia package REPL mode instruction (] add MOLE) to both julia/MOLE.jl/README.md and docs/src/index.md. Thank you!

@valeriabarra

Copy link
Copy Markdown
Collaborator

Hello @PandaHUN777 , thank you for your contribution.

I would like to let you know that we cannot accept contributions from anonymous users (i.e., users that do not have a full display name in their GitHub account). This is to ensure that our Authorship attribution and DOI-bearing MOLE library User Manual document that is uploaded on the public Zenodo archive are reflective of real people contributions.

@github-actions

Copy link
Copy Markdown

This pull request has had no activity for 14 days. It will be marked stale in 14 days if there is no further activity.

@github-actions github-actions Bot added the stale No recent activity; scheduled for automatic closure. label Oct 27, 2026
@github-actions

Copy link
Copy Markdown

Thanks for the discussion and work here. This pull request has had no activity for 28 days, so it is now marked stale. If there is no further activity, it will be closed in 42 days.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale No recent activity; scheduled for automatic closure.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MOLE.jl README updates

3 participants