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

feat(feature-flags): improve "IN/NOT_IN"; new rule actions #710

Merged
merged 6 commits into from
Oct 1, 2021

Conversation

gwlester
Copy link
Contributor

@gwlester gwlester commented Sep 29, 2021

Issue #, if available:

Description of changes:

Added new conditions for contains that clarify what they are testing.

Checklist

Breaking change checklist

RFC issue #699:

  • Migration process documented
  • Implement warnings (if it can live side by side)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.


View rendered docs/utilities/feature_flags.md

@boring-cyborg boring-cyborg bot added area/utilities documentation Improvements or additions to documentation tests labels Sep 29, 2021
@pull-request-size pull-request-size bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Sep 29, 2021
@gwlester gwlester changed the title Changes to "contains" logic to clarify and expand. feat(feature_flags): Changes to "contains" logic to clarify and expand. Sep 29, 2021
@codecov-commenter
Copy link

codecov-commenter commented Oct 1, 2021

Codecov Report

Merging #710 (8f0d26a) into develop (68e2c8e) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

❗ Current head 8f0d26a differs from pull request most recent head 6cfb9db. Consider uploading reports for the commit 6cfb9db to get more accurate results
Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #710      +/-   ##
===========================================
- Coverage    99.97%   99.93%   -0.05%     
===========================================
  Files          116      116              
  Lines         4866     4902      +36     
  Branches       267      271       +4     
===========================================
+ Hits          4865     4899      +34     
- Misses           0        1       +1     
- Partials         1        2       +1     
Impacted Files Coverage Δ
...owertools/utilities/feature_flags/feature_flags.py 100.00% <ø> (ø)
...ambda_powertools/utilities/feature_flags/schema.py 100.00% <100.00%> (ø)
aws_lambda_powertools/logging/logger.py 98.43% <0.00%> (-1.57%) ⬇️
aws_lambda_powertools/metrics/metrics.py 100.00% <0.00%> (ø)
aws_lambda_powertools/shared/constants.py 100.00% <0.00%> (ø)
aws_lambda_powertools/utilities/batch/sqs.py 100.00% <0.00%> (ø)
aws_lambda_powertools/shared/jmespath_utils.py 100.00% <0.00%> (ø)
aws_lambda_powertools/utilities/parameters/ssm.py 100.00% <0.00%> (ø)
..._lambda_powertools/utilities/feature_flags/base.py 100.00% <0.00%> (ø)
..._lambda_powertools/utilities/parameters/secrets.py 100.00% <0.00%> (ø)
... and 5 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 68e2c8e...6cfb9db. Read the comment docs.

@heitorlessa
Copy link
Contributor

Thanks a lot for helping expand the Feature flags utility @gwlester -- I find it much cleaner now.

Updating other areas like Schema docstring, and making a list of areas that we will deprecate when we go GA.

@heitorlessa heitorlessa added feature New feature or functionality area/feature_flags labels Oct 1, 2021
@heitorlessa heitorlessa changed the title feat(feature_flags): Changes to "contains" logic to clarify and expand. feat(feature-flags): improve "IN/NOT_IN"; new rule actions Oct 1, 2021
@heitorlessa heitorlessa merged commit c837e0a into aws-powertools:develop Oct 1, 2021
@boring-cyborg
Copy link

boring-cyborg bot commented Oct 1, 2021

Awesome work, congrats on your first merged pull request and thank you for helping improve everyone's experience!

heitorlessa added a commit to heitorlessa/aws-lambda-powertools-python that referenced this pull request Oct 1, 2021
…tools-python into develop

* 'develop' of https://github.com/awslabs/aws-lambda-powertools-python:
  feat(feature-flags): improve "IN/NOT_IN"; new rule actions (aws-powertools#710)
  feat(idempotency): makes customers unit testing easier (aws-powertools#719)
  feat(feature-flags): get_raw_configuration property in Store (aws-powertools#720)
heitorlessa added a commit to gwlester/aws-lambda-powertools-python that referenced this pull request Oct 1, 2021
* develop:
  feat(feature-flags): improve "IN/NOT_IN"; new rule actions (aws-powertools#710)
  feat(idempotency): makes customers unit testing easier (aws-powertools#719)
  feat(feature-flags): get_raw_configuration property in Store (aws-powertools#720)
  feat: boto3 sessions in batch, parameters & idempotency (aws-powertools#717)
  feat: add get_raw_configuration property in store; expose store
  fix(mypy): a few return types, type signatures, and untyped areas (aws-powertools#718)
  docs: Terraform reference for SAR Lambda Layer (aws-powertools#716)
  chore(deps-dev): bump flake8-bugbear from 21.9.1 to 21.9.2 (aws-powertools#712)
  chore(deps): bump boto3 from 1.18.49 to 1.18.51 (aws-powertools#713)
  fix(idempotency): sorting keys before hashing
@heitorlessa heitorlessa removed the documentation Improvements or additions to documentation label Oct 5, 2021
@gwlester gwlester deleted the feature-699 branch January 31, 2024 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or functionality size/L Denotes a PR that changes 100-499 lines, ignoring generated files. tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants