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

A Short Hike: Update installation instructions #3474

Merged
merged 6 commits into from
Jun 6, 2024

Conversation

chandler05
Copy link
Contributor

Updates the A Short Hike installation instructions to link more directly to the randomizer and clear up inconsistencies.

@github-actions github-actions bot added is: documentation Improvements or additions to documentation. waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. labels Jun 5, 2024
[Randomizer](https://github.com/BrandenEK/AShortHike.Randomizer/releases) zip, extract it, and move the contents
of the `Randomizer/` folder into your `Modding/` folder. After this step, your `Modding/` folder should have
`data/` and `plugins/` subfolders.
Open the [Randomizer Repository](https://github.com/BrandenEK/AShortHike.Randomizer) and follow
Copy link
Member

Choose a reason for hiding this comment

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

I'm a bit confused, this link goes to a repo that says "Installation Follow the instructions on the Mod Installer page for automatic or manual installation" So you're having the user go to a repo exclusively to go to another repo?

Copy link
Contributor

Choose a reason for hiding this comment

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

This is because the installer is now the one place where detailed installation instructions will be from now on. However, they are generic, and the Randomizer GitHub states which dependencies are also required to download

Copy link
Member

@Exempt-Medic Exempt-Medic left a comment

Choose a reason for hiding this comment

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

The wording changes look good, my question about the new structure was answered: it is intended.

@Exempt-Medic Exempt-Medic added waiting-on: core-review Issue/PR has been peer-reviewed and is ready to be merged or needs input from a core maintainer. and removed waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. labels Jun 6, 2024
@NewSoupVi NewSoupVi merged commit 223f2f5 into ArchipelagoMW:main Jun 6, 2024
11 checks passed
@chandler05 chandler05 deleted the patch-2 branch June 6, 2024 21:44
agilbert1412 pushed a commit to agilbert1412/Archipelago that referenced this pull request Jun 13, 2024
* A Short Hike: Update installation instructions

* Update setup_en.md

* Update setup_en.md

* Change link
jnschurig pushed a commit to Tranquilite0/Archipelago-SoulBlazer that referenced this pull request Jun 13, 2024
* A Short Hike: Update installation instructions

* Update setup_en.md

* Update setup_en.md

* Change link
sflavelle pushed a commit to sflavelle/Archipelago-tgc that referenced this pull request Jun 20, 2024
* A Short Hike: Update installation instructions

* Update setup_en.md

* Update setup_en.md

* Change link
qwint pushed a commit to qwint/Archipelago that referenced this pull request Jun 24, 2024
* A Short Hike: Update installation instructions

* Update setup_en.md

* Update setup_en.md

* Change link
AustinSumigray pushed a commit to AustinSumigray/Archipelago that referenced this pull request Jan 4, 2025
* A Short Hike: Update installation instructions

* Update setup_en.md

* Update setup_en.md

* Change link
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: documentation Improvements or additions to documentation. waiting-on: core-review Issue/PR has been peer-reviewed and is ready to be merged or needs input from a core maintainer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants