Skip to content

Conversation

@PatMyron
Copy link
Contributor

@PatMyron PatMyron requested a review from wata727 August 31, 2022 20:56
Copy link
Member

@wata727 wata727 left a comment

Choose a reason for hiding this comment

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

Thank you for working on this.

I agree that hard-coded lists are hard to maintain, but I believe such maintenance will be necessary to make this ruleset more valuable. I would like to maintain this list as necessary in the future, but what do you think?

@PatMyron
Copy link
Contributor Author

PatMyron commented Sep 6, 2022

@wata727 since there are currently ~1400 SDK-based rules and only a few dozen manual rules, I don't think it's worth actively maintaining this particular manual rule a couple times a year for every new AWS region. Pruning a couple rules like this can save us a lot of maintenance while the value of the ruleset stays roughly the same since it was just one of ~1500 rules

@wata727
Copy link
Member

wata727 commented Sep 7, 2022

Correct. But if we remove this rule for that reason, I think we will not be able to add manually-maintained rules.
I believe that lists like regions can be maintained by the community. I would like the ruleset to be a place where users bring value.

@PatMyron
Copy link
Contributor Author

PatMyron commented Sep 7, 2022

we will not be able to add manually-maintained rules

The churn is specific to this rule because there will be dozens of new AWS regions over time. None of the other manually maintained enums have nearly as many values, so the other manually maintained rules are fine in my opinion

maintained by the community

My concern is this rule would cause noise in the meantime until someone in the community steps up every time there's a new AWS region

@wata727
Copy link
Member

wata727 commented Sep 10, 2022

My concern is this rule would cause noise in the meantime until someone in the community steps up every time there's a new AWS region

Agreed, but I believe that not many people will be affected by this noise. It's not too late to remove this rule, even after many users reported it was causing inconvenience.

@PatMyron PatMyron merged commit 032c405 into master Oct 6, 2022
@PatMyron PatMyron deleted the region branch October 6, 2022 15:47
@PatMyron
Copy link
Contributor Author

PatMyron commented Oct 6, 2022

seeing multiple more AWS regions that look to be coming soon (ap-south-2/eu-south-2/eu-central-2), so merging this in anticipation of those region launches:

https://aws.amazon.com/blogs/aws/a-new-aws-region-opens-in-switzerland/
https://aws.amazon.com/blogs/aws/now-open-aws-region-in-spain/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants