fix(github): Ignore vulnerability alerts in FIXED or DISMISSED states - #14431

If it's too complicated to add GHE version detection to this, it would be better to undo this change and add the appropriate filtering on our client side instead, so it works with all.

We already know if we on GHE, so check should be easy. We can enable filter on GHE when we know on which GHE version it's available later.

But the problem this is addressing will still exist on GHE in the meantime?

If it's too complicated to add GHE version detection to this, it would be better to undo this change and add the appropriate filtering on our client side instead, so it works with all.

Indeed, I think it would be a bit complicated to check for the GHE versions that support of not the new state field.

But then, how would you filter on the client side? Because if we add the state field to the query, then it's also going to fail with the GHE versions that don't support it.

One option I thought about, but don't know if it's doable, is schema introspection, i.e. to check if the state field is present or not, and depending on the result include or not the state field in the query.

But the problem this is addressing will still exist on GHE in the meantime?

From what I understand, yes.

Reopening this, as I changed my mind :)

🎉 This PR is included in version 32.9.0 🎉

The release is available on:

• GitHub release
• 32.9.0

Your semantic-release bot 📦🚀

🎉 This issue has been resolved in version 32.76.3 🎉

The release is available on:

• GitHub release
• 32.76.3

Your semantic-release bot 📦🚀