Skip to content

Stardew Valley: 0.6.7 Day 1 fixes#6098

Open
agilbert1412 wants to merge 6 commits intoArchipelagoMW:mainfrom
agilbert1412:StardewValley/0.6.7_DayOneFixes
Open

Stardew Valley: 0.6.7 Day 1 fixes#6098
agilbert1412 wants to merge 6 commits intoArchipelagoMW:mainfrom
agilbert1412:StardewValley/0.6.7_DayOneFixes

Conversation

@agilbert1412
Copy link
Copy Markdown
Collaborator

What is this fixing or adding?

As usual, as soon as code is released and more eyes are put on it, some stuff is discovered.

  • The Shane and Sebastian Portrait filler items werent tagged properly.
  • The Beach Farm adds one secretsanity check, so it needs to be in the allsanity preset
  • The Allsanity preset is renamed to "Maxsanity" to encourage better defined terminology and Allsanity means something else
  • The "All Random" preset has been removed entirely. It has been the cause of too many beginner footguns over the years. People can still achieve the effect manually, but at least they'll have to try a little bit to ruin their own experience.

How was this tested?

Automated tests. Everything that was fixed, already "worked", it just wasn't working the way it should

@github-actions github-actions bot added the waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. label Apr 1, 2026
@agilbert1412 agilbert1412 requested a review from Jouramie April 1, 2026 23:03
@agilbert1412 agilbert1412 added the is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs. label Apr 1, 2026
@rosenothorns03
Copy link
Copy Markdown

I reported these bugs, so can confirm that they need fixing.

…he goal of doing ALL your checks" and "the concept of enabling as many checks as possible"
Copy link
Copy Markdown

@DavidWever DavidWever left a comment

Choose a reason for hiding this comment

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

Code looks good, simple fixes and test changes. Since everything passes this should be good to merge.

Copy link
Copy Markdown
Collaborator

@Jouramie Jouramie left a comment

Choose a reason for hiding this comment

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

It's do nice when the day 1 patch removes code lol

LGTM

@Jouramie Jouramie 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 Apr 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs. 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.

5 participants