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

I have an idea for a combo #119

Open
attickdoor opened this issue Dec 9, 2021 · 1 comment
Open

I have an idea for a combo #119

attickdoor opened this issue Dec 9, 2021 · 1 comment
Labels
documentation Improvements or additions to documentation good first issue Good for newcomers wontfix This will not be worked on

Comments

@attickdoor
Copy link
Collaborator

attickdoor commented Dec 9, 2021

It's probably not going to be added. Here are the requirements:

  1. The abilities in question must be truly mutually exclusive. If you can use A, there is literally no way to use B, and vice versa.
  • Caveat: If this isn't technically true, you need to have solid proof that there is NO contrived scenario in which you want to do things in a different way.
    • A common example is Cure and Cure 2. Cure has a faster cast time and costs less MP. So, I can easily contrive a use case where using Cure is preferable to using Cure 2. As such, there will never be a Cure combo.
  1. It actually saves you button space. Turning Inner Release into Fell Cleave when IR is active saves you no button spaces, because you need to press FC outside of IR anyways.
  2. It isn't wild and wacky, and the overall level of intelligence required to execute is about as low as the floor.

If you can pass all of these checks, there's a good chance it'll get added. If not, you're wasting your time by opening an issue. But I've already determined the large majority of cases that satisfy these requirements, so there are very few legitimate suggestions left.

@attickdoor attickdoor added documentation Improvements or additions to documentation good first issue Good for newcomers wontfix This will not be worked on labels Dec 9, 2021
@attickdoor attickdoor pinned this issue Dec 9, 2021
Repository owner deleted a comment from ArohkGit Dec 25, 2021
Repository owner deleted a comment from SoeyKitten Dec 25, 2021
Repository owner deleted a comment from mokanshu Dec 25, 2021
@attickdoor
Copy link
Collaborator Author

This is not the place to make suggestions. This is the place for you to read and understand that your suggestion is probably not valid.

Repository owner locked and limited conversation to collaborators Dec 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation good first issue Good for newcomers wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

1 participant