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

[script] [truffenyi-commune-quest] make truffenyi script more feature rich #6926

Merged
merged 9 commits into from
Oct 7, 2024

Conversation

vtcifer
Copy link
Contributor

@vtcifer vtcifer commented Sep 28, 2024

Handle solving the altar puzzle
Slightly better vial handling
Handle the shrine/ox puzzle

Still will not find shrine for you

@vtcifer vtcifer marked this pull request as draft September 28, 2024 19:46
drink the vial if you find it (with warning about how long this can take)
otherwise assume it's already been drunk like before
some rubocop clean up
@vtcifer
Copy link
Contributor Author

vtcifer commented Sep 29, 2024

GTG even with rubocop warnings (most are indent issue which are present for better code collapse)

@vtcifer vtcifer marked this pull request as ready for review September 29, 2024 16:46
@vtcifer
Copy link
Contributor Author

vtcifer commented Sep 29, 2024

GTG even with rubocop warnings (most are indent issue which are present for better code collapse)

@MahtraDR
Copy link
Collaborator

MahtraDR commented Sep 30, 2024

Can uuu you please do the rubocop autocorrects or propose a change to the rubocop rules? I accept the indentation folding idea, but rubocop keeps me sane too.

@vtcifer
Copy link
Contributor Author

vtcifer commented Oct 6, 2024

IMO rubocop makes a good sniff test for happy flows - it catches a lot of 'oops' in those. But for unhappy flows it should be 'someone 4-eyes this', not as the final arbiter. If the committer can articulate a good justification for the ignoring it should be OK.

Just my .02

*ETA: That said, I'll take a pass at inline rubocop directives explaining this in code.

@vtcifer
Copy link
Contributor Author

vtcifer commented Oct 6, 2024

Given how messy this PR is with the commits, just ping me on discord if you're OK with how I've handled rubocop and I'll do another branch with a cleaner commit history for merging.

@MahtraDR MahtraDR merged commit 3e29d01 into elanthia-online:master Oct 7, 2024
2 checks passed
@MahtraDR
Copy link
Collaborator

MahtraDR commented Oct 7, 2024

This works for me. Thanks!

@vtcifer vtcifer deleted the patch-2 branch October 7, 2024 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants