-
Notifications
You must be signed in to change notification settings - Fork 243
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
ci(dependabot): Ignore patch versions #3373
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
It's too noisy currently and since we support ranges in our constraints only a minor/major version bump could potentially impact our customers. And if a patch bump does fix a security concern, we would expect that to be handled through the dependabot security advisories which is separate from this configuration.
Equartey
approved these changes
Jul 7, 2023
Jordan-Nelson
approved these changes
Jul 7, 2023
khatruong2009
pushed a commit
to khatruong2009/amplify-flutter
that referenced
this pull request
Jul 24, 2023
It's too noisy currently and since we support ranges in our constraints only a minor/major version bump could potentially impact our customers. And if a patch bump does fix a security concern, we would expect that to be handled through the dependabot security advisories which is separate from this configuration.
khatruong2009
pushed a commit
to khatruong2009/amplify-flutter
that referenced
this pull request
Jul 24, 2023
It's too noisy currently and since we support ranges in our constraints only a minor/major version bump could potentially impact our customers. And if a patch bump does fix a security concern, we would expect that to be handled through the dependabot security advisories which is separate from this configuration.
khatruong2009
added a commit
to khatruong2009/amplify-flutter
that referenced
this pull request
Jul 24, 2023
author Kha Truong <[email protected]> 1687902177 -0700 committer Kha Truong <[email protected]> 1690235526 -0700 parent b12c326 author Kha Truong <[email protected]> 1687902177 -0700 committer Kha Truong <[email protected]> 1690235203 -0700 parent b12c326 author Kha Truong <[email protected]> 1687902177 -0700 committer Kha Truong <[email protected]> 1690234697 -0700 parent b12c326 author Kha Truong <[email protected]> 1687902177 -0700 committer Kha Truong <[email protected]> 1690234586 -0700 included a parameter to let the developer set a default country code to the authenticator. still need to audit country codes and test for phone username changed variable names to make them more consistent throughout files removed a line of commented code removed default dial code from the example file rename the Country class to CountryCode to reduce confusion Changed the CountryCode Class on the country_code.dart file switch name of the class from CountryCode to DialCode change country_code file name to dial code additional countryCode -> dialCode changes fixed imports in other files and made the constructor for dialCode private removed the mention of country from dial_code.dart file, made defaultDialCode the second layer under dialCodeOptions for future additions, deprecated country resolver class ci(canaries): More improvements (aws-amplify#3320) - Switches to `macos-13` for iOS canaries (seems to be more reliable) - Adds pre-build to Android to prevent timeouts - Adds cache removal for iOS launcher action to improve flakiness ci(dependabot): Ignore patch versions (aws-amplify#3373) It's too noisy currently and since we support ranges in our constraints only a minor/major version bump could potentially impact our customers. And if a patch bump does fix a security concern, we would expect that to be handled through the dependabot security advisories which is separate from this configuration. ci(dependabot): Fix kotlin name (aws-amplify#3385) Should be `kotlin_version` I think given recent dependabot PRs fix(codegen): Mark Cognito map as sparse (aws-amplify#3386) * fix(codegen): Mark Cognito map as `sparse` Cognito incorrectly labels the challenge parameters response type as `Map<String, String>` but returns `null` for values in certain instances. We introduce a workaround to mark the type as `sparse` until the service model is fixed. * chore(auth): Regenerate SDK chore(deps): Bump pigeon to `^10.0.0` (aws-amplify#3250) chore(deps): Bump pigeon to `^10.1.2` chore(repo): Improve analysis speed (aws-amplify#3387) ci(dependabot): Ignore Kotlin gradle plugin (aws-amplify#3402) For the same reason we ignore the Kotlin version itself chore(api): Improve WS logging (aws-amplify#3401) Improves logging in the WebSocketBloc so that events have metadata logged as well. Since this is currently done with `debug` log level, there should be no new information exposed in production logs. Update packages/authenticator/amplify_authenticator/lib/amplify_authenticator.dart Co-authored-by: Dillon Nys <[email protected]> Update packages/authenticator/amplify_authenticator/lib/src/mixins/authenticator_phone_field.dart Co-authored-by: Dillon Nys <[email protected]> changed _countriesResolver name to _dialCodeResolver and fixed references added deprecation and typedef to avoid breaking change with Country Resolver class name change added template and macro for dialCodeOptions comment chore: deprecated use of the word country and fixed references in example app and dial_code.dart files chore: removed initialDialCode variable. It is already set in the constructor fix: changed areas to dialCode name to more accurately represent variable fix: changed oldCountry to oldDialCode to remove country fix: changed country to dialCode for consistency fix: changed dialCode to an enum and modified variables to support that Update packages/authenticator/amplify_authenticator/lib/src/utils/dial_code.dart Co-authored-by: Jordan Nelson <[email protected]> fix: changed American Samoa dial code to match dial code resolver key fix: removed areas in favor of dialCodes and fixed cascading fix: change country to dialCode and change sort to sortedBy fix: made this.defaultDialCode required in authenticator state. Also removed two unnecessary instances of setting defaultDialCode (already set in the DialCodeOptions constructor) Update packages/authenticator/amplify_authenticator/lib/src/state/authenticator_state.dart Co-authored-by: Dillon Nys <[email protected]> fix: remove sortedBy cascade and remove necessary toList operation Update packages/authenticator/amplify_authenticator/lib/src/l10n/auth_strings_resolver.dart Co-authored-by: Dillon Nys <[email protected]> included a parameter to let the developer set a default country code to the authenticator. still need to audit country codes and test for phone username removed default dial code from the example file Changed the CountryCode Class on the country_code.dart file change country_code file name to dial code added template and macro for dialCodeOptions comment fix: changed American Samoa dial code to match dial code resolver key fix: change country to dialCode and change sort to sortedBy fix: made this.defaultDialCode required in authenticator state. Also removed two unnecessary instances of setting defaultDialCode (already set in the DialCodeOptions constructor) Update packages/authenticator/amplify_authenticator/lib/src/state/authenticator_state.dart Co-authored-by: Dillon Nys <[email protected]> fix: remove sortedBy cascade and remove necessary toList operation Update packages/authenticator/amplify_authenticator/lib/src/l10n/auth_strings_resolver.dart Co-authored-by: Dillon Nys <[email protected]>
khatruong2009
added a commit
that referenced
this pull request
Jul 25, 2023
* parent b12c326 author Kha Truong <[email protected]> 1687902177 -0700 committer Kha Truong <[email protected]> 1690235526 -0700 parent b12c326 author Kha Truong <[email protected]> 1687902177 -0700 committer Kha Truong <[email protected]> 1690235203 -0700 parent b12c326 author Kha Truong <[email protected]> 1687902177 -0700 committer Kha Truong <[email protected]> 1690234697 -0700 parent b12c326 author Kha Truong <[email protected]> 1687902177 -0700 committer Kha Truong <[email protected]> 1690234586 -0700 included a parameter to let the developer set a default country code to the authenticator. still need to audit country codes and test for phone username changed variable names to make them more consistent throughout files removed a line of commented code removed default dial code from the example file rename the Country class to CountryCode to reduce confusion Changed the CountryCode Class on the country_code.dart file switch name of the class from CountryCode to DialCode change country_code file name to dial code additional countryCode -> dialCode changes fixed imports in other files and made the constructor for dialCode private removed the mention of country from dial_code.dart file, made defaultDialCode the second layer under dialCodeOptions for future additions, deprecated country resolver class ci(canaries): More improvements (#3320) - Switches to `macos-13` for iOS canaries (seems to be more reliable) - Adds pre-build to Android to prevent timeouts - Adds cache removal for iOS launcher action to improve flakiness ci(dependabot): Ignore patch versions (#3373) It's too noisy currently and since we support ranges in our constraints only a minor/major version bump could potentially impact our customers. And if a patch bump does fix a security concern, we would expect that to be handled through the dependabot security advisories which is separate from this configuration. ci(dependabot): Fix kotlin name (#3385) Should be `kotlin_version` I think given recent dependabot PRs fix(codegen): Mark Cognito map as sparse (#3386) * fix(codegen): Mark Cognito map as `sparse` Cognito incorrectly labels the challenge parameters response type as `Map<String, String>` but returns `null` for values in certain instances. We introduce a workaround to mark the type as `sparse` until the service model is fixed. * chore(auth): Regenerate SDK chore(deps): Bump pigeon to `^10.0.0` (#3250) chore(deps): Bump pigeon to `^10.1.2` chore(repo): Improve analysis speed (#3387) ci(dependabot): Ignore Kotlin gradle plugin (#3402) For the same reason we ignore the Kotlin version itself chore(api): Improve WS logging (#3401) Improves logging in the WebSocketBloc so that events have metadata logged as well. Since this is currently done with `debug` log level, there should be no new information exposed in production logs. Update packages/authenticator/amplify_authenticator/lib/amplify_authenticator.dart Co-authored-by: Dillon Nys <[email protected]> Update packages/authenticator/amplify_authenticator/lib/src/mixins/authenticator_phone_field.dart Co-authored-by: Dillon Nys <[email protected]> changed _countriesResolver name to _dialCodeResolver and fixed references added deprecation and typedef to avoid breaking change with Country Resolver class name change added template and macro for dialCodeOptions comment chore: deprecated use of the word country and fixed references in example app and dial_code.dart files chore: removed initialDialCode variable. It is already set in the constructor fix: changed areas to dialCode name to more accurately represent variable fix: changed oldCountry to oldDialCode to remove country fix: changed country to dialCode for consistency fix: changed dialCode to an enum and modified variables to support that Update packages/authenticator/amplify_authenticator/lib/src/utils/dial_code.dart Co-authored-by: Jordan Nelson <[email protected]> fix: changed American Samoa dial code to match dial code resolver key fix: removed areas in favor of dialCodes and fixed cascading fix: change country to dialCode and change sort to sortedBy fix: made this.defaultDialCode required in authenticator state. Also removed two unnecessary instances of setting defaultDialCode (already set in the DialCodeOptions constructor) Update packages/authenticator/amplify_authenticator/lib/src/state/authenticator_state.dart Co-authored-by: Dillon Nys <[email protected]> fix: remove sortedBy cascade and remove necessary toList operation Update packages/authenticator/amplify_authenticator/lib/src/l10n/auth_strings_resolver.dart Co-authored-by: Dillon Nys <[email protected]> included a parameter to let the developer set a default country code to the authenticator. still need to audit country codes and test for phone username removed default dial code from the example file Changed the CountryCode Class on the country_code.dart file change country_code file name to dial code added template and macro for dialCodeOptions comment fix: changed American Samoa dial code to match dial code resolver key fix: change country to dialCode and change sort to sortedBy fix: made this.defaultDialCode required in authenticator state. Also removed two unnecessary instances of setting defaultDialCode (already set in the DialCodeOptions constructor) Update packages/authenticator/amplify_authenticator/lib/src/state/authenticator_state.dart Co-authored-by: Dillon Nys <[email protected]> fix: remove sortedBy cascade and remove necessary toList operation Update packages/authenticator/amplify_authenticator/lib/src/l10n/auth_strings_resolver.dart Co-authored-by: Dillon Nys <[email protected]> * chore: squashed commits from previous work * Revert "Merge branch 'feat/auth/countrycode' of https://github.com/khatruong2009/amplify-flutter into feat/auth/countrycode" This reverts commit a3bcd71, reversing changes made to 21c4656. --------- Co-authored-by: Dillon Nys <[email protected]>
dnys1
added a commit
that referenced
this pull request
Jul 27, 2023
It's too noisy currently and since we support ranges in our constraints only a minor/major version bump could potentially impact our customers. And if a patch bump does fix a security concern, we would expect that to be handled through the dependabot security advisories which is separate from this configuration.
dnys1
added a commit
that referenced
this pull request
Jul 27, 2023
* parent b12c326 author Kha Truong <[email protected]> 1687902177 -0700 committer Kha Truong <[email protected]> 1690235526 -0700 parent b12c326 author Kha Truong <[email protected]> 1687902177 -0700 committer Kha Truong <[email protected]> 1690235203 -0700 parent b12c326 author Kha Truong <[email protected]> 1687902177 -0700 committer Kha Truong <[email protected]> 1690234697 -0700 parent b12c326 author Kha Truong <[email protected]> 1687902177 -0700 committer Kha Truong <[email protected]> 1690234586 -0700 included a parameter to let the developer set a default country code to the authenticator. still need to audit country codes and test for phone username changed variable names to make them more consistent throughout files removed a line of commented code removed default dial code from the example file rename the Country class to CountryCode to reduce confusion Changed the CountryCode Class on the country_code.dart file switch name of the class from CountryCode to DialCode change country_code file name to dial code additional countryCode -> dialCode changes fixed imports in other files and made the constructor for dialCode private removed the mention of country from dial_code.dart file, made defaultDialCode the second layer under dialCodeOptions for future additions, deprecated country resolver class ci(canaries): More improvements (#3320) - Switches to `macos-13` for iOS canaries (seems to be more reliable) - Adds pre-build to Android to prevent timeouts - Adds cache removal for iOS launcher action to improve flakiness ci(dependabot): Ignore patch versions (#3373) It's too noisy currently and since we support ranges in our constraints only a minor/major version bump could potentially impact our customers. And if a patch bump does fix a security concern, we would expect that to be handled through the dependabot security advisories which is separate from this configuration. ci(dependabot): Fix kotlin name (#3385) Should be `kotlin_version` I think given recent dependabot PRs fix(codegen): Mark Cognito map as sparse (#3386) * fix(codegen): Mark Cognito map as `sparse` Cognito incorrectly labels the challenge parameters response type as `Map<String, String>` but returns `null` for values in certain instances. We introduce a workaround to mark the type as `sparse` until the service model is fixed. * chore(auth): Regenerate SDK chore(deps): Bump pigeon to `^10.0.0` (#3250) chore(deps): Bump pigeon to `^10.1.2` chore(repo): Improve analysis speed (#3387) ci(dependabot): Ignore Kotlin gradle plugin (#3402) For the same reason we ignore the Kotlin version itself chore(api): Improve WS logging (#3401) Improves logging in the WebSocketBloc so that events have metadata logged as well. Since this is currently done with `debug` log level, there should be no new information exposed in production logs. Update packages/authenticator/amplify_authenticator/lib/amplify_authenticator.dart Co-authored-by: Dillon Nys <[email protected]> Update packages/authenticator/amplify_authenticator/lib/src/mixins/authenticator_phone_field.dart Co-authored-by: Dillon Nys <[email protected]> changed _countriesResolver name to _dialCodeResolver and fixed references added deprecation and typedef to avoid breaking change with Country Resolver class name change added template and macro for dialCodeOptions comment chore: deprecated use of the word country and fixed references in example app and dial_code.dart files chore: removed initialDialCode variable. It is already set in the constructor fix: changed areas to dialCode name to more accurately represent variable fix: changed oldCountry to oldDialCode to remove country fix: changed country to dialCode for consistency fix: changed dialCode to an enum and modified variables to support that Update packages/authenticator/amplify_authenticator/lib/src/utils/dial_code.dart Co-authored-by: Jordan Nelson <[email protected]> fix: changed American Samoa dial code to match dial code resolver key fix: removed areas in favor of dialCodes and fixed cascading fix: change country to dialCode and change sort to sortedBy fix: made this.defaultDialCode required in authenticator state. Also removed two unnecessary instances of setting defaultDialCode (already set in the DialCodeOptions constructor) Update packages/authenticator/amplify_authenticator/lib/src/state/authenticator_state.dart Co-authored-by: Dillon Nys <[email protected]> fix: remove sortedBy cascade and remove necessary toList operation Update packages/authenticator/amplify_authenticator/lib/src/l10n/auth_strings_resolver.dart Co-authored-by: Dillon Nys <[email protected]> included a parameter to let the developer set a default country code to the authenticator. still need to audit country codes and test for phone username removed default dial code from the example file Changed the CountryCode Class on the country_code.dart file change country_code file name to dial code added template and macro for dialCodeOptions comment fix: changed American Samoa dial code to match dial code resolver key fix: change country to dialCode and change sort to sortedBy fix: made this.defaultDialCode required in authenticator state. Also removed two unnecessary instances of setting defaultDialCode (already set in the DialCodeOptions constructor) Update packages/authenticator/amplify_authenticator/lib/src/state/authenticator_state.dart Co-authored-by: Dillon Nys <[email protected]> fix: remove sortedBy cascade and remove necessary toList operation Update packages/authenticator/amplify_authenticator/lib/src/l10n/auth_strings_resolver.dart Co-authored-by: Dillon Nys <[email protected]> * chore: squashed commits from previous work * Revert "Merge branch 'feat/auth/countrycode' of https://github.com/khatruong2009/amplify-flutter into feat/auth/countrycode" This reverts commit a3bcd71, reversing changes made to 21c4656. --------- Co-authored-by: Dillon Nys <[email protected]>
dnys1
added a commit
that referenced
this pull request
Jul 28, 2023
It's too noisy currently and since we support ranges in our constraints only a minor/major version bump could potentially impact our customers. And if a patch bump does fix a security concern, we would expect that to be handled through the dependabot security advisories which is separate from this configuration.
dnys1
added a commit
that referenced
this pull request
Jul 28, 2023
* parent b12c326 author Kha Truong <[email protected]> 1687902177 -0700 committer Kha Truong <[email protected]> 1690235526 -0700 parent b12c326 author Kha Truong <[email protected]> 1687902177 -0700 committer Kha Truong <[email protected]> 1690235203 -0700 parent b12c326 author Kha Truong <[email protected]> 1687902177 -0700 committer Kha Truong <[email protected]> 1690234697 -0700 parent b12c326 author Kha Truong <[email protected]> 1687902177 -0700 committer Kha Truong <[email protected]> 1690234586 -0700 included a parameter to let the developer set a default country code to the authenticator. still need to audit country codes and test for phone username changed variable names to make them more consistent throughout files removed a line of commented code removed default dial code from the example file rename the Country class to CountryCode to reduce confusion Changed the CountryCode Class on the country_code.dart file switch name of the class from CountryCode to DialCode change country_code file name to dial code additional countryCode -> dialCode changes fixed imports in other files and made the constructor for dialCode private removed the mention of country from dial_code.dart file, made defaultDialCode the second layer under dialCodeOptions for future additions, deprecated country resolver class ci(canaries): More improvements (#3320) - Switches to `macos-13` for iOS canaries (seems to be more reliable) - Adds pre-build to Android to prevent timeouts - Adds cache removal for iOS launcher action to improve flakiness ci(dependabot): Ignore patch versions (#3373) It's too noisy currently and since we support ranges in our constraints only a minor/major version bump could potentially impact our customers. And if a patch bump does fix a security concern, we would expect that to be handled through the dependabot security advisories which is separate from this configuration. ci(dependabot): Fix kotlin name (#3385) Should be `kotlin_version` I think given recent dependabot PRs fix(codegen): Mark Cognito map as sparse (#3386) * fix(codegen): Mark Cognito map as `sparse` Cognito incorrectly labels the challenge parameters response type as `Map<String, String>` but returns `null` for values in certain instances. We introduce a workaround to mark the type as `sparse` until the service model is fixed. * chore(auth): Regenerate SDK chore(deps): Bump pigeon to `^10.0.0` (#3250) chore(deps): Bump pigeon to `^10.1.2` chore(repo): Improve analysis speed (#3387) ci(dependabot): Ignore Kotlin gradle plugin (#3402) For the same reason we ignore the Kotlin version itself chore(api): Improve WS logging (#3401) Improves logging in the WebSocketBloc so that events have metadata logged as well. Since this is currently done with `debug` log level, there should be no new information exposed in production logs. Update packages/authenticator/amplify_authenticator/lib/amplify_authenticator.dart Co-authored-by: Dillon Nys <[email protected]> Update packages/authenticator/amplify_authenticator/lib/src/mixins/authenticator_phone_field.dart Co-authored-by: Dillon Nys <[email protected]> changed _countriesResolver name to _dialCodeResolver and fixed references added deprecation and typedef to avoid breaking change with Country Resolver class name change added template and macro for dialCodeOptions comment chore: deprecated use of the word country and fixed references in example app and dial_code.dart files chore: removed initialDialCode variable. It is already set in the constructor fix: changed areas to dialCode name to more accurately represent variable fix: changed oldCountry to oldDialCode to remove country fix: changed country to dialCode for consistency fix: changed dialCode to an enum and modified variables to support that Update packages/authenticator/amplify_authenticator/lib/src/utils/dial_code.dart Co-authored-by: Jordan Nelson <[email protected]> fix: changed American Samoa dial code to match dial code resolver key fix: removed areas in favor of dialCodes and fixed cascading fix: change country to dialCode and change sort to sortedBy fix: made this.defaultDialCode required in authenticator state. Also removed two unnecessary instances of setting defaultDialCode (already set in the DialCodeOptions constructor) Update packages/authenticator/amplify_authenticator/lib/src/state/authenticator_state.dart Co-authored-by: Dillon Nys <[email protected]> fix: remove sortedBy cascade and remove necessary toList operation Update packages/authenticator/amplify_authenticator/lib/src/l10n/auth_strings_resolver.dart Co-authored-by: Dillon Nys <[email protected]> included a parameter to let the developer set a default country code to the authenticator. still need to audit country codes and test for phone username removed default dial code from the example file Changed the CountryCode Class on the country_code.dart file change country_code file name to dial code added template and macro for dialCodeOptions comment fix: changed American Samoa dial code to match dial code resolver key fix: change country to dialCode and change sort to sortedBy fix: made this.defaultDialCode required in authenticator state. Also removed two unnecessary instances of setting defaultDialCode (already set in the DialCodeOptions constructor) Update packages/authenticator/amplify_authenticator/lib/src/state/authenticator_state.dart Co-authored-by: Dillon Nys <[email protected]> fix: remove sortedBy cascade and remove necessary toList operation Update packages/authenticator/amplify_authenticator/lib/src/l10n/auth_strings_resolver.dart Co-authored-by: Dillon Nys <[email protected]> * chore: squashed commits from previous work * Revert "Merge branch 'feat/auth/countrycode' of https://github.com/khatruong2009/amplify-flutter into feat/auth/countrycode" This reverts commit a3bcd71, reversing changes made to 21c4656. --------- Co-authored-by: Dillon Nys <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
It's too noisy currently and since we support ranges in our constraints only a minor/major version bump could potentially impact our customers. And if a patch bump does fix a security concern, we would expect that to be handled through the dependabot security advisories which is separate from this configuration.