Merged the latest develop code into main #2831
Conversation
Bumps [lint](https://github.com/passsy/dart-lint) from 2.2.0 to 2.3.0. - [Release notes](https://github.com/passsy/dart-lint/releases) - [Changelog](https://github.com/passsy/dart-lint/blob/master/CHANGELOG.md) - [Commits](passsy/dart-lint@v2.2.0...v2.3.0) --- updated-dependencies: - dependency-name: lint dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…undation#2272) Bumps [syncfusion_flutter_calendar](https://github.com/syncfusion/flutter-widgets/tree/master/packages) from 23.2.7 to 24.1.41. - [Release notes](https://github.com/syncfusion/flutter-widgets/releases) - [Commits](https://github.com/syncfusion/flutter-widgets/commits/HEAD/packages) --- updated-dependencies: - dependency-name: syncfusion_flutter_calendar dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…doesFoundation#2268) * Feature request: Implement User Profile Modification Feature * changes * changes * proper dynamic call * fixing tests * fixing coverage * writing test for missing lines * writing test for missing lines * writing test for missing lines * writing test for missing lines * writing test for missing lines * adding requested changes * adding requested changes * adding requested changes * adding requested changes * adding requested changes * adding requested changes * adding requested changes * test for missing lines * adding requested changes * adding requested changes * test for missing lines
* resolved conflicts * fetchmore result typecast * added ImageService and written its tests * added coverage to missing lines. * Minor fix * fixed failing test * added ImageService and written its tests * resolved requested changes. * added changes to locator.dart * add debug print. * resolve requested changes. * resolved requested changes.
…wa (PalisadoesFoundation#2266) * testing automation * testing automation * Streamlining CI/CD workflow for iOS Build and Release Process in Talawa * Streamlining CI/CD workflow for iOS Build and Release Process in Talawa * changes * changes * changes
* added check for all translation file * fix falling test * fix the python style guide * made the requested changes * now the script will report for every file if there is error * revert back the double quotes to single quotes * error message will show path of file * improved the output message * used namedtuple to improve readability
* added check for all translation file * fix falling test * fix the python style guide * made the requested changes * now the script will report for every file if there is error * revert back the double quotes to single quotes * error message will show path of file * improved the output message * used namedtuple to improve readability * used argparser * made the changes * made the changes
…sadoesFoundation#2284) * Verification of Translation Comparison Script Functionality * testing when default language has 1 extra key * testing when default language has 1 extra key * testing when key in DL has different spacing in a key * testing multiple extra keys in DL * testing when key is wrong in other translation files * Modifying script to give more detailed description for the missing or mismatched key * testing * testing * testing * testing * more detailed description for error * testing * testing if new script displays the correct error for new key that is not present in non default files * testing for wrong key in non default * modified compare translation script * testing * testing * modified
* Updated Code * Updated Code
* test for access request screen * fixed falling test cases * fix falling test * fix falling test
* modify fetchRegistrantsByEvent test * test for dispose method * test for getters * fix format issues * fix format issues * fix codebase issues * add test for getEvents
) Bumps [cached_network_image](https://github.com/Baseflow/flutter_cached_network_image) from 3.3.0 to 3.3.1. - [Commits](Baseflow/flutter_cached_network_image@v3.3.0...v3.3.1) --- updated-dependencies: - dependency-name: cached_network_image dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…sFoundation#2294) Bumps [syncfusion_flutter_datepicker](https://github.com/syncfusion/flutter-widgets/tree/master/packages) from 24.1.41 to 24.1.43. - [Release notes](https://github.com/syncfusion/flutter-widgets/releases) - [Commits](https://github.com/syncfusion/flutter-widgets/commits/HEAD/packages) --- updated-dependencies: - dependency-name: syncfusion_flutter_datepicker dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ndation#2292) Bumps [flutter_local_notifications](https://github.com/MaikuB/flutter_local_notifications) from 16.2.0 to 16.3.0. - [Release notes](https://github.com/MaikuB/flutter_local_notifications/releases) - [Commits](MaikuB/flutter_local_notifications@flutter_local_notifications-v16.2.0...flutter_local_notifications-v16.3.0) --- updated-dependencies: - dependency-name: flutter_local_notifications dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…oundation#2293) Bumps [syncfusion_flutter_calendar](https://github.com/syncfusion/flutter-widgets/tree/master/packages) from 24.1.41 to 24.1.43. - [Release notes](https://github.com/syncfusion/flutter-widgets/releases) - [Commits](https://github.com/syncfusion/flutter-widgets/commits/HEAD/packages) --- updated-dependencies: - dependency-name: syncfusion_flutter_calendar dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
… Filters UI (PalisadoesFoundation#2245) * Add:Filters * Add:Documentation * Add:MultiLingual Support * Fix:Format * Reduced: Lines of Codes * Fixed:Tests * Add:Fix Failing test case * Add:Tests for requested lines * Add:test for filter changing * Fixed:changes requested * Add:Changes * Fix:Documentation * Fix:Dart version * Fixed:spacing * Remove: useless docs * Remove:Spaces * Removed:spaces * Fix:Constant Sizes * Fix:Format issues * Minor FIx * Fix:format * Fix:Dynamic size * Fix:Dynamic size * Fix:Failing test * Fix:Failing test
…alisadoesFoundation#2290) * Comprehensive Language Translation Enhancement for App Globalization * adding more translations * testing * test * test * test * proper format for translation files * formating * formating * formating * fixing translation * fixing linting errors * writing test for missing lines * writing test for missing lines * writing test for missing lines * fixing translations * fixing translations * fixing translations * fixing translations
…lisadoesFoundation#2302) Bumps [tj-actions/changed-files](https://github.com/tj-actions/changed-files) from 35 to 41. - [Release notes](https://github.com/tj-actions/changed-files/releases) - [Changelog](https://github.com/tj-actions/changed-files/blob/main/HISTORY.md) - [Commits](tj-actions/changed-files@v35...v41) --- updated-dependencies: - dependency-name: tj-actions/changed-files dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* created settings page * formatted the code * fixed failing tests * resolved requested changes * fixed failing tests
* Created Test * Updated Formatting
* fixed events fetching bug * fixed failing tests * added comments to event_queries * fixed failing test
…undation#2306) * Refactor: Decouple ViewModel by Moving UI Logic to View * refactoring app_settings_view_model and app_seetings_page * writing test for missing lines
…2257) * feature: Removed all references to Google firebase * formatted firebase_mocks.dart * removed ignore directive for custom lint rule * formatted login_view_model file * removal of commented code * removed fcmToken test * avoid dynamic calls check passed * format check * fixed linting issue * added documentation for event_queries file * format check
…oundation#2638) Bumps [syncfusion_flutter_calendar](https://github.com/syncfusion/flutter-widgets/tree/master/packages) from 27.1.57 to 27.1.58. - [Release notes](https://github.com/syncfusion/flutter-widgets/releases) - [Commits](https://github.com/syncfusion/flutter-widgets/commits/HEAD/packages) --- updated-dependencies: - dependency-name: syncfusion_flutter_calendar dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ndation#2641) Bumps [flutter_local_notifications](https://github.com/MaikuB/flutter_local_notifications) from 18.0.0 to 18.0.1. - [Release notes](https://github.com/MaikuB/flutter_local_notifications/releases) - [Commits](MaikuB/flutter_local_notifications@flutter_local_notifications-v18.0.0...flutter_local_notifications-v18.0.1) --- updated-dependencies: - dependency-name: flutter_local_notifications dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Fundraising Campaigns * adding tests * adding tests * adding tests * adding tests * adding tests * adding tests * adding tests
* added auto-label.json and updated issues.yml * named the issues.yml to issue.yml * improved the sync of issue.yml
* 20250109191235 Deleted all files in the develop branch in anticipation of merging develop-postgres into develop cleanly * 20250109191239 Merge develop-postgres into develop
…lisadoesFoundation#2772) Co-authored-by: Satyam Jha <satyamj210@gmail.com>
…undation#2828) * 20250518155938 Deleted all files in the develop branch in anticipation of merging develop-postgres into develop cleanly * 20250518155958 Merge develop-postgres into develop * Saving before reset 05/18/25 16:07:14 * Saving before reset 05/18/25 16:07:19 * Saving before reset 05/18/25 16:10:31 * Saving before reset 05/18/25 16:13:42
…undation#2829) * 20250518171810 Deleted all files in the develop branch in anticipation of merging develop-postgres into develop cleanly * 20250518171845 Merge develop-postgres into develop * 20250518172320 Merge develop-postgres into develop
* 20250518175350 Fixed old develop-postgres references * 20250518175537 Fixed old develop-postgres references
…n of merging develop-postgres into main cleanly
…n of merging develop-postgres into main cleanly
palisadoes
added
the
ignore-sensitive-files-pr
WalkthroughThis update introduces new and revised workflow automation, configuration, and compliance scripts for the project. Major changes include new and updated GitHub Actions workflows for PR validation, Python code style enforcement, and documentation checks; upgrades to Android and Flutter build configurations; new and revised linting and ignore rules; and the removal or relocation of several scripts and documentation files. Changes
Sequence Diagram(s)sequenceDiagram
participant Developer
participant GitHub Actions
participant CodeRabbit
participant Python Compliance Scripts
Developer->>GitHub Actions: Push or open PR
GitHub Actions->>GitHub Actions: Run workflows (build, test, lint, etc.)
GitHub Actions->>Python Compliance Scripts: Run docstring, lint, and translation checks
Python Compliance Scripts-->>GitHub Actions: Report compliance status
GitHub Actions->>CodeRabbit: Request review (if configured)
CodeRabbit-->>GitHub Actions: Submit review/approval
GitHub Actions->>GitHub Actions: Validate CodeRabbit approval (pull-request-review.yml)
GitHub Actions-->>Developer: Allow or block merge based on approval/compliance
Possibly related PRs
Suggested reviewers
Note ⚡️ AI Code Reviews for VS Code, Cursor, WindsurfCodeRabbit now has a plugin for VS Code, Cursor and Windsurf. This brings AI code reviews directly in the code editor. Each commit is reviewed immediately, finding bugs before the PR is raised. Seamless context handoff to your AI code agent ensures that you can easily incorporate review feedback. Note ⚡️ Faster reviews with cachingCodeRabbit now supports caching for code and dependencies, helping speed up reviews. This means quicker feedback, reduced wait times, and a smoother review experience overall. Cached data is encrypted and stored securely. This feature will be automatically enabled for all accounts on May 16th. To opt out, configure ✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
Our Pull Request Approval ProcessThanks for contributing! Testing Your CodeRemember, your PRs won't be reviewed until these criteria are met:
Our policies make our code better. ReviewersDo not assign reviewers. Our Queue Monitors will review your PR and assign them.
Reviewing Your CodeYour reviewer(s) will have the following roles:
Other🎯 Please be considerate of our volunteers' time. Contacting the person who assigned the reviewers is not advised unless they ask for your input. Do not @ the person who did the assignment otherwise. |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 38
🔭 Outside diff range comments (1)
devtools_options.yaml (1)
1-4: 🧹 Nitpick (assertive)Clarify empty
extensionsmapping.
Currentlyextensions:has no entries and will map tonull. If the intention is to represent an empty mapping, consider explicitly usingextensions: {}or adding a commented placeholder for future extension configurations.
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
⛔ Files ignored due to path filters (262)
INSTALLATION.mdis excluded by!*.mdassets/images/icon _like_.svgis excluded by!**/*.svgdocs/blog/2021-08-26-welcome/docusaurus-plushie-banner.jpegis excluded by!**/*.jpegdocs/docs/auto-docs/__404error.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme-class-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme-class.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/AppTheme.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/blackPrimary-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/blackSecondary-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/blackTertiary-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/blue-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/bodyText1.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/bodyText2.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/bodyText3.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/button.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/green-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/grey-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/headline1.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/headline2.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/headline3.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/headline4.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/headline5.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/headline6.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/lightGrey-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/overline.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/primary-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/red-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/secondary-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/shadow-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/subtitle1.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/subtitle2.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/tertiary-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/title.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/white-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/AppTheme/yellow-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/apptheme-library-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/apptheme-library.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/apptheme/index.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors-class-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors-class.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/TalawaErrors.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/chatMessageDeletionFailed-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/chatMessageSendFailed-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/emailAccountPresent-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/eventCreationFailed-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/eventDeletionFailed-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/eventUpdateFailed-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/failedToDetermineProject-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/memberRequestExist-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/organizationNotFound-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/postCreationFailed-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/postDeletionFailed-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/postUpdateFailed-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/refreshAccessTokenExpiredException-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/userActionNotSaved-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/userNotAuthenticated-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/userNotFound-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/userProfileDeletionFailed-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/userProfileUpdateFailed-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/wrongCredentials-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/youAreOfflineUnableToLogin-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/youAreOfflineUnableToLogout-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/TalawaErrors/youAreOfflineUnableToSignUp-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/constants_app_strings-library-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/constants_app_strings-library.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_app_strings/index.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_constants/HiveKeys-class-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_constants/HiveKeys-class.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_constants/HiveKeys/HiveKeys.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_constants/HiveKeys/asymetricKeyBoxKey-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_constants/HiveKeys/eventFeedKey-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_constants/HiveKeys/offlineActionQueueKey-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_constants/HiveKeys/orgBoxKey-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_constants/HiveKeys/pluginBoxKey-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_constants/HiveKeys/postFeedKey-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_constants/HiveKeys/urlBoxKey-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_constants/HiveKeys/userBoxKey-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_constants/constants_constants-library-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_constants/constants_constants-library.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_constants/index.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_constants/languages.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_constants/supportedCurrencies.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_custom_theme/TalawaTheme-class-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_custom_theme/TalawaTheme-class.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_custom_theme/TalawaTheme/TalawaTheme.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_custom_theme/TalawaTheme/darkTheme.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_custom_theme/TalawaTheme/lightTheme.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_custom_theme/constants_custom_theme-library-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_custom_theme/constants_custom_theme-library.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_custom_theme/index.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_quick_actions/ShortCutMenu-class-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_quick_actions/ShortCutMenu-class.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_quick_actions/ShortCutMenu/ShortCutMenu.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_quick_actions/ShortCutMenu/chatAction-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_quick_actions/ShortCutMenu/eventAction-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_quick_actions/ShortCutMenu/feedAction-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_quick_actions/ShortCutMenu/quickActionsList.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_quick_actions/constants_quick_actions-library-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_quick_actions/constants_quick_actions-library.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_quick_actions/index.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/EventEndTypes-class-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/EventEndTypes-class.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/EventEndTypes/EventEndTypes.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/EventEndTypes/after-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/EventEndTypes/never-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/EventEndTypes/on-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/EventIntervals-class-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/EventIntervals-class.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/EventIntervals/EventIntervals.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/EventIntervals/daily-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/EventIntervals/monthly-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/EventIntervals/weekly-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/EventIntervals/yearly-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/Frequency-class-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/Frequency-class.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/Frequency/Frequency.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/Frequency/daily-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/Frequency/monthly-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/Frequency/weekly-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/Frequency/yearly-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/RecurrenceUtils-class-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/RecurrenceUtils-class.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/RecurrenceUtils/RecurrenceUtils.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/RecurrenceUtils/getRecurrenceRuleText.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/RecurrenceUtils/getWeekDayOccurenceInMonth.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/RecurrenceUtils/getWeekDaysString.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/RecurrenceUtils/isLastOccurenceOfWeekDay.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/RecurrenceUtils/monthNames-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/RecurrenceUtils/weekDayOccurences-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/RecurrenceUtils/weekDays-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/RecurringEventMutationType-class-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/RecurringEventMutationType-class.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/RecurringEventMutationType/RecurringEventMutationType.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/RecurringEventMutationType/allInstances-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/RecurringEventMutationType/thisAndFollowingInstances-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/RecurringEventMutationType/thisInstance-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/WeekDays-class-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/WeekDays-class.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/WeekDays/WeekDays.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/WeekDays/friday-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/WeekDays/monday-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/WeekDays/saturday-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/WeekDays/sunday-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/WeekDays/thursday-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/WeekDays/tuesday-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/WeekDays/wednesday-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/constants_recurrence_values-library-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/constants_recurrence_values-library.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/days.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_recurrence_values/index.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes-class-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes-class.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/Routes.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/addPostScreen-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/addTask-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/appSettings-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/calendar-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/chatMessageScreen-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/createEventPage-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/customRecurrencePage-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/demoExploreEventsScreen-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/demoHomeScreen-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/demoPageViewRoute-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/demoProfilePage-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/editEventPage-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/editProfilePage-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/editTask-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/eventInfoPage-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/eventTasks-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/exploreEventsScreen-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/homeScreen-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/individualPost-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/joinOrg-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/languageSelectionRoute-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/loginScreen-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/mainScreen-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/manageVolunteerGroup-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/orgInfoScreen-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/pinnedPostPage-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/pinnedPostScreen-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/profilePage-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/progressDialog-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/recoverScreen-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/requestAccess-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/selectContact-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/selectOrgScreen-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/setUrlScreen-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/signupDetailScreen-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/splashScreen-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/updateScreen-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/userTasks-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/volunteerGroupScreen-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/Routes/waitingScreen-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/constants_routing_constants-library-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/constants_routing_constants-library.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_routing_constants/index.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_timeout/TimeOuts-class-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_timeout/TimeOuts-class.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_timeout/TimeOuts/TimeOuts.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_timeout/TimeOuts/large-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_timeout/TimeOuts/mediums-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_timeout/TimeOuts/small-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_timeout/constants_timeout-library-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_timeout/constants_timeout-library.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/constants_timeout/index.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_language_icon/LanguageIcon-class-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_language_icon/LanguageIcon-class.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_language_icon/LanguageIcon/LanguageIcon.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_language_icon/LanguageIcon/paint.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_language_icon/LanguageIcon/shouldRepaint.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_language_icon/custom_painters_language_icon-library-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_language_icon/custom_painters_language_icon-library.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_language_icon/index.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_talawa_logo/AppLogo-class-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_talawa_logo/AppLogo-class.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_talawa_logo/AppLogo/AppLogo.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_talawa_logo/AppLogo/paint.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_talawa_logo/AppLogo/shouldRepaint.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_talawa_logo/custom_painters_talawa_logo-library-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_talawa_logo/custom_painters_talawa_logo-library.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_talawa_logo/index.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_telegram_logo/TelegramLogo-class-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_telegram_logo/TelegramLogo-class.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_telegram_logo/TelegramLogo/TelegramLogo.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_telegram_logo/TelegramLogo/paint.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_telegram_logo/TelegramLogo/shouldRepaint.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_telegram_logo/custom_painters_telegram_logo-library-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_telegram_logo/custom_painters_telegram_logo-library.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_telegram_logo/index.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_whatsapp_logo/WhatsappLogo-class-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_whatsapp_logo/WhatsappLogo-class.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_whatsapp_logo/WhatsappLogo/WhatsappLogo.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_whatsapp_logo/WhatsappLogo/paint.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_whatsapp_logo/WhatsappLogo/shouldRepaint.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_whatsapp_logo/custom_painters_whatsapp_logo-library-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_whatsapp_logo/custom_painters_whatsapp_logo-library.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/custom_painters_whatsapp_logo/index.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/demo_server_data_direct_chat_demo_data/demo_server_data_direct_chat_demo_data-library-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/demo_server_data_direct_chat_demo_data/demo_server_data_direct_chat_demo_data-library.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/demo_server_data_direct_chat_demo_data/directChatDemoData-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/demo_server_data_direct_chat_demo_data/index.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/demo_server_data_events_demo_data/demo_server_data_events_demo_data-library-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/demo_server_data_events_demo_data/demo_server_data_events_demo_data-library.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/demo_server_data_events_demo_data/eventsDemoData-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/demo_server_data_events_demo_data/index.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/demo_server_data_pinned_post_demo_data/demo_server_data_pinned_post_demo_data-library-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/demo_server_data_pinned_post_demo_data/demo_server_data_pinned_post_demo_data-library.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/demo_server_data_pinned_post_demo_data/index.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/demo_server_data_pinned_post_demo_data/pinnedPostsDemoData-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/demo_server_data_post_demo_data/demo_server_data_post_demo_data-library-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/demo_server_data_post_demo_data/demo_server_data_post_demo_data-library.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/demo_server_data_post_demo_data/index.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/demo_server_data_post_demo_data/postsDemoData-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/enums_enums/ActionType-enum-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/enums_enums/ActionType.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/enums_enums/ActionType/values-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/enums_enums/CachedOperationType-enum-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/enums_enums/CachedOperationType.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/enums_enums/CachedOperationType/values-constant.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/enums_enums/CachedOperationTypeAdapter-class-sidebar.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/enums_enums/CachedOperationTypeAdapter-class.mdis excluded by!**/docs/docs/**docs/docs/auto-docs/enums_enums/CachedOperationTypeAdapter/CachedOperationTypeAdapter.mdis excluded by!**/docs/docs/**
📒 Files selected for processing (38)
.coderabbit.yaml(1 hunks).flake8(1 hunks).github/dependabot.yaml(1 hunks).github/pull_request_template.md(1 hunks).github/workflows/archive/documentationcheck.py(1 hunks).github/workflows/archive/talawa_mobile_md_mdx_format_adjuster.py(1 hunks).github/workflows/check_ignore.py(0 hunks).github/workflows/compare_translations.py(0 hunks).github/workflows/documentationcheck.py(0 hunks).github/workflows/pull-request-review.yml(1 hunks).github/workflows/pull-request-target.yml(1 hunks).github/workflows/pull-request.yml(11 hunks).github/workflows/push-deploy-website.yml(3 hunks).github/workflows/push.yml(8 hunks).github/workflows/requirements.txt(1 hunks).github/workflows/scripts/check_docstrings.py(1 hunks).github/workflows/scripts/check_ignore.py(1 hunks).github/workflows/scripts/compare_translations.py(1 hunks).github/workflows/scripts/countline.py(1 hunks).github/workflows/scripts/validate-coderabbit.sh(0 hunks).github/workflows/talawa_mobile_md_mdx_format_adjuster.py(0 hunks).gitignore(3 hunks).husky/pre-commit(1 hunks).pydocstyle(1 hunks)CODEOWNERS(1 hunks)analysis_options.yaml(1 hunks)android/app/build.gradle(4 hunks)android/app/proguard-rules.pro(1 hunks)android/build.gradle(1 hunks)android/gradle/wrapper/gradle-wrapper.properties(1 hunks)android/settings.gradle(1 hunks)devtools_options.yaml(1 hunks)docs/blog/2019-05-28-first-blog-post.md(0 hunks)docs/blog/2019-05-29-long-blog-post.md(0 hunks)docs/blog/2021-08-01-mdx-blog-post.mdx(0 hunks)docs/blog/2021-08-26-welcome/index.md(0 hunks)docs/blog/authors.yml(0 hunks)docs/blog/tags.yml(0 hunks)
💤 Files with no reviewable changes (11)
- docs/blog/2019-05-29-long-blog-post.md
- docs/blog/2019-05-28-first-blog-post.md
- docs/blog/tags.yml
- docs/blog/authors.yml
- docs/blog/2021-08-01-mdx-blog-post.mdx
- docs/blog/2021-08-26-welcome/index.md
- .github/workflows/talawa_mobile_md_mdx_format_adjuster.py
- .github/workflows/documentationcheck.py
- .github/workflows/compare_translations.py
- .github/workflows/scripts/validate-coderabbit.sh
- .github/workflows/check_ignore.py
🧰 Additional context used
🧠 Learnings (1)
.github/workflows/pull-request.yml (1)
Learnt from: VanshikaSabharwal
PR: PalisadoesFoundation/talawa#0
File: :0-0
Timestamp: 2025-03-31T09:18:01.805Z
Learning: In the PalisadoesFoundation/talawa repository, the "Base branch check" job in the GitHub Actions workflow is designed to be skipped when PRs are correctly targeting the develop-postgres branch, which is the expected behavior.
🪛 YAMLlint (1.37.1)
.github/workflows/push-deploy-website.yml
[error] 43-43: trailing spaces
(trailing-spaces)
.coderabbit.yaml
[error] 22-22: trailing spaces
(trailing-spaces)
.github/workflows/push.yml
[warning] 38-38: too few spaces before comment: expected 2
(comments)
[warning] 43-43: too few spaces before comment: expected 2
(comments)
[warning] 90-90: too few spaces before comment: expected 2
(comments)
[warning] 95-95: too few spaces before comment: expected 2
(comments)
[warning] 122-122: too few spaces before comment: expected 2
(comments)
[warning] 127-127: too few spaces before comment: expected 2
(comments)
[warning] 160-160: too few spaces before comment: expected 2
(comments)
.github/workflows/pull-request.yml
[warning] 27-27: missing starting space in comment
(comments)
[warning] 36-36: too few spaces before comment: expected 2
(comments)
[warning] 131-131: too few spaces before comment: expected 2
(comments)
[error] 134-134: too many spaces before colon
(colons)
[error] 134-134: trailing spaces
(trailing-spaces)
[error] 163-163: trailing spaces
(trailing-spaces)
[error] 164-164: trailing spaces
(trailing-spaces)
[warning] 184-184: too few spaces before comment: expected 2
(comments)
[warning] 184-184: missing starting space in comment
(comments)
🪛 Ruff (0.11.9)
.github/workflows/scripts/check_ignore.py
10-10: Missing return type annotation for private function _arg_parser_resolver
(ANN202)
53-53: Missing return type annotation for private function _filepaths_in_directories
(ANN202)
53-53: Missing type annotation for function argument directories
(ANN001)
76-76: Missing return type annotation for private function _check_for_ignore_directive
Add return type annotation: bool
(ANN202)
76-76: Missing type annotation for function argument filePath
(ANN001)
123-123: Unnecessary mode argument
Remove mode argument
(UP015)
192-192: Trailing comma missing
Add trailing comma
(COM812)
200-200: Missing return type annotation for public function main
Add return type annotation: None
(ANN201)
214-214: subprocess call: check for execution of untrusted input
(S603)
214-214: Starting a process with a partial executable path
(S607)
214-214: Trailing comma missing
Add trailing comma
(COM812)
218-218: subprocess call: check for execution of untrusted input
(S603)
218-218: Starting a process with a partial executable path
(S607)
224-224: subprocess call: check for execution of untrusted input
(S603)
225-225: Starting a process with a partial executable path
(S607)
225-225: Trailing comma missing
Add trailing comma
(COM812)
244-244: Trailing comma missing
Add trailing comma
(COM812)
251-251: Trailing comma missing
Add trailing comma
(COM812)
.github/workflows/scripts/compare_translations.py
17-17: Trailing comma missing
Add trailing comma
(COM812)
21-21: Missing return type annotation for public function compare_translations
(ANN201)
22-22: Missing type annotation for function argument default_translation
(ANN001)
22-22: Missing type annotation for function argument other_translation
(ANN001)
22-22: Missing type annotation for function argument default_file
(ANN001)
22-22: Missing type annotation for function argument other_file
(ANN001)
22-22: Trailing comma missing
Add trailing comma
(COM812)
59-59: Missing return type annotation for public function load_translation
(ANN201)
59-59: Missing type annotation for function argument filepath
(ANN001)
68-68: Unnecessary mode argument
Remove mode argument
(UP015)
70-70: Unnecessary assignment to translation before return statement
Remove unnecessary assignment
(RET504)
73-73: Missing return type annotation for public function check_translations
Add return type annotation: None
(ANN201)
73-73: Missing type annotation for function argument directory
(ANN001)
84-84: Trailing comma missing
Add trailing comma
(COM812)
115-115: Missing return type annotation for public function main
Add return type annotation: None
(ANN201)
130-130: Trailing comma missing
Add trailing comma
(COM812)
144-144: Trailing comma missing
Add trailing comma
(COM812)
.github/workflows/archive/talawa_mobile_md_mdx_format_adjuster.py
1-1: Shebang is present but file is not executable
(EXE001)
20-20: Missing return type annotation for public function escape_mdx_characters
(ANN201)
20-20: Missing type annotation for function argument text
(ANN001)
45-45: Missing return type annotation for public function adjust_dart_code_blocks
(ANN201)
45-45: Missing type annotation for function argument text
(ANN001)
63-63: Unnecessary assignment to text before return statement
Remove unnecessary assignment
(RET504)
66-66: Missing return type annotation for public function process_file
Add return type annotation: None
(ANN201)
66-66: Missing type annotation for function argument filepath
(ANN001)
78-78: Unnecessary mode argument
Remove mode argument
(UP015)
91-91: Missing return type annotation for public function main
Add return type annotation: None
(ANN201)
104-104: Trailing comma missing
Add trailing comma
(COM812)
.github/workflows/archive/documentationcheck.py
52-52: Missing return type annotation for private function _arg_parser_resolver
(ANN202)
89-89: Unnecessary assignment to result before return statement
Remove unnecessary assignment
(RET504)
92-92: Missing return type annotation for public function check_for_documentation
(ANN201)
92-92: Missing type annotation for function argument diff_item
(ANN001)
146-146: Missing return type annotation for public function main
Add return type annotation: None
(ANN201)
165-165: Use f-string instead of format call
Convert to f-string
(UP032)
166-166: Use f-string instead of format call
Convert to f-string
(UP032)
204-204: Trailing comma missing
Add trailing comma
(COM812)
211-211: Trailing comma missing
Add trailing comma
(COM812)
216-216: Trailing comma missing
Add trailing comma
(COM812)
.github/workflows/scripts/check_docstrings.py
14-14: Missing return type annotation for public function validate_docstring
(ANN201)
14-14: Missing type annotation for function argument file_path
(ANN001)
30-30: Unnecessary mode argument
Remove mode argument
(UP015)
33-33: Do not catch blind exception: Exception
(BLE001)
53-54: Use a single if statement instead of nested if statements
(SIM102)
67-67: Trailing comma missing
Add trailing comma
(COM812)
76-76: Missing return type annotation for public function ignore_function
(ANN201)
76-76: Missing type annotation for function argument function
(ANN001)
76-76: Missing type annotation for function argument file_path
(ANN001)
100-100: match_arguments_to_docstring is too complex (11 > 10)
(C901)
100-100: Missing return type annotation for public function match_arguments_to_docstring
(ANN201)
100-100: Missing type annotation for function argument function
(ANN001)
100-100: Missing type annotation for function argument docstring
(ANN001)
100-100: Missing type annotation for function argument line_number
(ANN001)
129-129: Trailing comma missing
Add trailing comma
(COM812)
138-138: Loop control variable argument_function not used within loop body
(B007)
152-152: Trailing comma missing
Add trailing comma
(COM812)
162-162: Loop control variable argument_docstring not used within loop body
(B007)
176-176: Trailing comma missing
Add trailing comma
(COM812)
187-187: Missing return type annotation for public function function_has_decorator
(ANN201)
187-187: Missing type annotation for function argument start
(ANN001)
187-187: Missing type annotation for function argument lines
(ANN001)
209-209: Missing return type annotation for public function decorator_in_docstring_exception_list
(ANN201)
209-209: Missing type annotation for function argument item
(ANN001)
240-240: Missing return type annotation for public function extract_function_arguments
(ANN201)
240-240: Missing type annotation for function argument start
(ANN001)
240-240: Missing type annotation for function argument lines
(ANN001)
262-262: Unnecessary elif after continue statement
Remove unnecessary elif
(RET507)
283-283: Unnecessary assignment to result before return statement
Remove unnecessary assignment
(RET504)
286-286: extract_docstring is too complex (12 > 10)
(C901)
286-286: Missing return type annotation for public function extract_docstring
(ANN201)
286-286: Missing type annotation for function argument func_name
(ANN001)
286-286: Missing type annotation for function argument line_number
(ANN001)
286-286: Missing type annotation for function argument lines
(ANN001)
303-303: Trailing comma missing
Add trailing comma
(COM812)
339-339: Trailing comma missing
Add trailing comma
(COM812)
356-356: Do not catch blind exception: Exception
(BLE001)
364-364: Trailing comma missing
Add trailing comma
(COM812)
369-369: Trailing comma missing
Add trailing comma
(COM812)
376-376: Trailing comma missing
Add trailing comma
(COM812)
386-386: Trailing comma missing
Add trailing comma
(COM812)
399-399: Trailing comma missing
Add trailing comma
(COM812)
410-410: Trailing comma missing
Add trailing comma
(COM812)
421-421: Unnecessary assignment to result before return statement
Remove unnecessary assignment
(RET504)
424-424: Missing return type annotation for public function evaluate_docstring_description
(ANN201)
424-424: Missing type annotation for function argument func_name
(ANN001)
424-424: Missing type annotation for function argument docstring_start
(ANN001)
424-424: Missing type annotation for function argument parser
(ANN001)
451-451: Trailing comma missing
Add trailing comma
(COM812)
464-464: Trailing comma missing
Add trailing comma
(COM812)
470-470: Missing return type annotation for public function evaluate_docstring_args
(ANN201)
470-470: Missing type annotation for function argument func_name
(ANN001)
470-470: Missing type annotation for function argument docstring_start
(ANN001)
470-470: Missing type annotation for function argument docstring
(ANN001)
470-470: Missing type annotation for function argument parser
(ANN001)
485-485: Trailing comma missing
Add trailing comma
(COM812)
493-493: Unnecessary else after return statement
Remove unnecessary else
(RET505)
503-503: Trailing comma missing
Add trailing comma
(COM812)
515-515: Trailing comma missing
Add trailing comma
(COM812)
530-530: Trailing comma missing
Add trailing comma
(COM812)
541-541: Trailing comma missing
Add trailing comma
(COM812)
550-550: Unnecessary assignment to result before return statement
Remove unnecessary assignment
(RET504)
553-553: Missing return type annotation for public function evaluate_docstring_returns
(ANN201)
553-553: Missing type annotation for function argument func_name
(ANN001)
553-553: Missing type annotation for function argument docstring_start
(ANN001)
553-553: Missing type annotation for function argument docstring
(ANN001)
553-553: Missing type annotation for function argument parser
(ANN001)
579-579: Trailing comma missing
Add trailing comma
(COM812)
598-598: Trailing comma missing
Add trailing comma
(COM812)
610-610: Trailing comma missing
Add trailing comma
(COM812)
623-623: Trailing comma missing
Add trailing comma
(COM812)
639-639: Trailing comma missing
Add trailing comma
(COM812)
651-651: Trailing comma missing
Add trailing comma
(COM812)
658-658: Missing return type annotation for public function is_docstring_delimiter
(ANN201)
658-658: Missing type annotation for function argument line
(ANN001)
670-670: Trailing comma missing
Add trailing comma
(COM812)
672-672: Unnecessary assignment to result before return statement
Remove unnecessary assignment
(RET504)
675-675: Missing return type annotation for public function check_directory
(ANN201)
675-675: Missing type annotation for function argument directory
(ANN001)
675-675: Missing type annotation for function argument exclude_dirs
(ANN001)
715-715: Missing return type annotation for public function main
Add return type annotation: None
(ANN201)
760-760: Trailing comma missing
Add trailing comma
(COM812)
766-766: Trailing comma missing
Add trailing comma
(COM812)
🪛 actionlint (1.7.7)
.github/workflows/pull-request.yml
60-60: "github.head_ref" is potentially untrusted. avoid using it directly in inline scripts. instead, pass it through an environment variable. see https://docs.github.com/en/actions/security-for-github-actions/security-guides/security-hardening-for-github-actions for more details
(expression)
196-196: property "changed_files" is not defined in object type {changed-files: {conclusion: string; outcome: string; outputs: {string => string}}}
(expression)
197-197: property "changed_files" is not defined in object type {changed-files: {conclusion: string; outcome: string; outputs: {string => string}}}
(expression)
197-197: property "changed_files" is not defined in object type {changed-files: {conclusion: string; outcome: string; outputs: {string => string}}}
(expression)
317-317: the runner of "actions/setup-python@v4" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: Testing codebase
🔇 Additional comments (22)
.github/workflows/scripts/countline.py (1)
17-18: List indentation is valid
The adjustment of the numbered list’s indentation under the docstring aligns with Pydocstyle conventions and is consistent with the rest of the documentation. No further changes needed here.android/gradle/wrapper/gradle-wrapper.properties (1)
6-6: Approve Gradle wrapper version bump to 8.9
Aligns with AGP 8.7.1 and modern build environments.android/settings.gradle (1)
22-22: Approve Android Gradle plugin version bump to 8.7.1
This update aligns with the Gradle wrapper upgrade and SDK version increases, ensuring compatibility with modern Android tooling.android/app/build.gradle (4)
36-36: Approve compileSdkVersion upgrade to 35
Raising the compile SDK to 35 aligns with the target SDK bump and leverages the latest Android APIs.
45-47: Approve minSdk and targetSdk version updates
UpdatingminSdkVersionto 22 andtargetSdkVersionto 35 modernizes your device support baseline.
45-47: Verify impact of minSdk increase
RaisingminSdkVersionfrom 21 to 22 drops support for some older devices. Please confirm this aligns with your user base or analytics data.
56-79: Approve core library desugaring configuration
ThecoreLibraryDesugaringEnabled truesetting together withcoreLibraryDesugaring 'com.android.tools:desugar_jdk_libs:2.0.3'allows using modern Java APIs on legacy Android platforms.android/build.gradle (1)
11-25: Approve consistent namespace and compile SDK configuration
The newafterEvaluateblock enforces a defaultnamespaceandcompileSdkVersion 35across all Android subprojects, ensuring uniform build settings..pydocstyle (1)
1-3: Configuration appears correct.
The.pydocstylesettings enforce the Google docstring convention and properly ignore D415 and D205. No changes required..flake8 (1)
1-3: Flake8 configuration is sound.
The ignored rules align with Black formatting and project conventions, and the max line length of 80 is appropriate..github/dependabot.yaml (1)
10-10: Confirm Dependabot target branch update
Thetarget-branchhas been changed to"develop". Ensure this aligns with the intended branch strategy and that other workflows (e.g., PR templates) target the same branch to avoid inconsistencies..github/workflows/pull-request-target.yml (1)
22-25: Corrected action version and input parameter
Upgrading tothollander/actions-comment-pull-request@v3and using thecomment-tagparameter is appropriate for the latest action syntax..gitignore (1)
146-152: Ignore Python environment directories and files
The added entries (.env,.venv,env/,venv/,ENV/,env.bak/,venv.bak/) correctly exclude common Python virtual environment and environment variable files from version control..husky/pre-commit (1)
1-17: Well-structured pre-commit hook for Dart documentation generationThis pre-commit hook is well-implemented with proper error handling and conditional execution. It effectively checks for Dart file changes, runs documentation generation, and automatically stages the generated files.
A few things that work well here:
- Proper error handling that aborts commits when doc generation fails
- Clear messaging to indicate what's happening
- Conditional execution that skips unnecessary processing when no Dart files changed
.github/workflows/push-deploy-website.yml (2)
17-19: Correctly updated branch and path filtersThe workflow now targets the 'develop' branch with a specific path filter for docs directory, which properly limits workflow execution to relevant changes.
37-37: Environment name correctly aligned with branch nameThe environment name update to 'develop' aligns with the branch targeting change, maintaining consistency.
.coderabbit.yaml (3)
2-3: Documentation link addedAdding a link to the documentation is helpful for developers who want to learn more about configuration options.
6-7: Chat configuration moved for better organizationMoving the chat section to the top with auto_reply enabled is a good organizational change.
9-10: Review profile changed to be more thoroughChanging from "chill" to "assertive" profile will provide more thorough code reviews, and disabling poem generation focuses reviews on technical content.
.github/workflows/push.yml (2)
53-70: Improved error handling for uncommitted changesThe updated script path and added logic to stash/restore uncommitted changes to pubspec.lock provides better handling of workflow execution when there are local modifications.
108-108: Improved environment variable referenceThe reference to CODECOV_UNIQUE_NAME has been updated to use the proper env variable context.
.github/workflows/pull-request-review.yml (1)
1-42: LGTM! Well-structured GitHub workflow for CodeRabbit review validation.The workflow is well-designed to validate that CodeRabbit has reviewed and approved PRs before they can be merged. The implementation is clean and follows best practices for GitHub Actions.
| @@ -10,15 +9,15 @@ | |||
| This script was created to help improve code quality by encouraging | |||
| contributors to create reusable code. | |||
|
|
|||
| NOTE: | |||
| Other: | |||
There was a problem hiding this comment.
🧹 Nitpick (assertive)
Clarify header section name
The new section header “Other:” is a bit vague—readers might not immediately understand that it refers to the coding/documentation standards the script complies with. Consider renaming it to something more descriptive, for example “Standards Compliance:” or “Supported Tools:”, to make its intent clearer.
🤖 Prompt for AI Agents
In .github/workflows/scripts/countline.py at line 12, rename the section header
currently labeled "Other:" to a more descriptive title such as "Standards
Compliance:" or "Supported Tools:" to clearly indicate that this section refers
to the coding or documentation standards the script complies with.
| @@ -3,4 +3,4 @@ distributionBase=GRADLE_USER_HOME | |||
| distributionPath=wrapper/dists | |||
| zipStoreBase=GRADLE_USER_HOME | |||
| zipStorePath=wrapper/dists | |||
| distributionUrl=https\://services.gradle.org/distributions/gradle-7.6.1-all.zip | |||
| distributionUrl=https\://services.gradle.org/distributions/gradle-8.9-all.zip | |||
There was a problem hiding this comment.
🧹 Nitpick (assertive)
Document required Java version for Gradle 8.9
Gradle 8.9 mandates JDK 17 or higher. Please update the README or developer documentation to specify this prerequisite for local and CI builds.
🤖 Prompt for AI Agents
In android/gradle/wrapper/gradle-wrapper.properties at line 6, the Gradle
version was updated to 8.9, which requires JDK 17 or higher. Update the README
or relevant developer documentation to clearly state that JDK 17 or above is
required for local development and CI builds to ensure compatibility with Gradle
8.9.
| # Please add these rules to your existing keep rules in order to suppress warnings. | ||
| # This is generated automatically by the Android Gradle plugin. | ||
| -dontwarn org.bouncycastle.jce.provider.BouncyCastleProvider | ||
| -dontwarn org.bouncycastle.pqc.jcajce.provider.BouncyCastlePQCProvider |
There was a problem hiding this comment.
Missing ProGuard rules inclusion in build configuration
The new proguard-rules.pro file isn't referenced in the app module’s build.gradle (e.g., via
proguardFiles(
getDefaultProguardFile('proguard-android-optimize.txt'),
'proguard-rules.pro'
)under buildTypes.release), so these rules won’t be applied at build time.
🤖 Prompt for AI Agents
In android/app/proguard-rules.pro lines 1 to 4, the ProGuard rules are defined
but not applied because the app module's build.gradle file does not reference
this file. To fix this, open the app module's build.gradle and under the
buildTypes.release section, add or update the proguardFiles configuration to
include 'proguard-rules.pro' alongside the default ProGuard file. This ensures
the rules in proguard-rules.pro are applied during the release build.
| @@ -19,7 +19,7 @@ pluginManagement { | |||
|
|
|||
| plugins { | |||
| id "dev.flutter.flutter-plugin-loader" version "1.0.2" | |||
| id "com.android.application" version "7.3.0" apply false | |||
| id "com.android.application" version '8.7.1' apply false | |||
There was a problem hiding this comment.
🧹 Nitpick (assertive)
Maintain consistent quoting for plugin versions
Other plugin declarations use double quotes for version literals. For consistency, consider:
-id "com.android.application" version '8.7.1' apply false
+id "com.android.application" version "8.7.1" apply false📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| id "com.android.application" version '8.7.1' apply false | |
| id "com.android.application" version "8.7.1" apply false |
🤖 Prompt for AI Agents
In android/settings.gradle at line 22, the plugin version is enclosed in single
quotes while other plugin declarations use double quotes. Change the single
quotes around the version '8.7.1' to double quotes to maintain consistent
quoting style across the file.
| dependencies { | ||
| implementation 'com.android.support:multidex:1.0.3' | ||
| implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:1.8.10" | ||
| coreLibraryDesugaring 'com.android.tools:desugar_jdk_libs:2.0.3' | ||
| } |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Migrate multidex dependency to AndroidX
Replace the legacy Support Library artifact with the AndroidX equivalent to avoid potential conflicts:
dependencies {
- implementation 'com.android.support:multidex:1.0.3'
+ implementation 'androidx.multidex:multidex:2.0.1'
}📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| dependencies { | |
| implementation 'com.android.support:multidex:1.0.3' | |
| implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:1.8.10" | |
| coreLibraryDesugaring 'com.android.tools:desugar_jdk_libs:2.0.3' | |
| } | |
| dependencies { | |
| implementation 'androidx.multidex:multidex:2.0.1' | |
| implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:1.8.10" | |
| coreLibraryDesugaring 'com.android.tools:desugar_jdk_libs:2.0.3' | |
| } |
🤖 Prompt for AI Agents
In android/app/build.gradle around lines 76 to 80, the multidex dependency uses
the legacy Support Library artifact 'com.android.support:multidex:1.0.3'.
Replace this with the AndroidX equivalent 'androidx.multidex:multidex:2.0.1' to
ensure compatibility and avoid conflicts. Update the implementation line
accordingly while keeping the other dependencies unchanged.
| for argument_function in arguments_function: | ||
| # Track whether the argument is defined | ||
| # in the docstring parameters | ||
| for argument_docstring in arguments_docstring: | ||
| if argument_docstring not in arguments_function: | ||
| violations.append( | ||
| Violation( | ||
| line=line_number + 1, | ||
| function=function.name, | ||
| issue=f"""\ | ||
| Argument '{argument_docstring}' defined in the docstring is not \ | ||
| an argument in the function""", | ||
| action=f"""\ | ||
| Remove argument '{argument_docstring}' from the docstring""", | ||
| ) | ||
| ) | ||
| bad_argument_function = True | ||
| break | ||
| if bad_argument_function: | ||
| break | ||
|
|
There was a problem hiding this comment.
🛠️ Refactor suggestion
Complex & incorrect argument matching logic
Nested loops create O(n²) work and the inner if argument_docstring not in arguments_function: test never uses the outer-loop variable, causing false positives/negatives.
Replace both nested loops with set operations:
- for argument_function in arguments_function:
- ...
- # We found an error, no need to continue generating violations
- if not bad_argument_function:
- ...
+ missing_in_doc = set(arguments_function) - set(arguments_docstring)
+ extra_in_doc = set(arguments_docstring) - set(arguments_function)
+
+ for item in missing_in_doc:
+ violations.append(
+ Violation(
+ line=line_number + 1,
+ function=function.name,
+ issue=f"Argument '{item}' is not documented in the docstring.",
+ action=f"Add argument '{item}' to the 'Args:' section.",
+ )
+ )
+ for item in extra_in_doc:
+ violations.append(
+ Violation(
+ line=line_number + 1,
+ function=function.name,
+ issue=f"Argument '{item}' documented but not present in signature.",
+ action=f"Remove '{item}' from the docstring.",
+ )
+ )This is clearer, faster, and avoids the loop-control-variable warnings.
Committable suggestion skipped: line range outside the PR's diff.
🧰 Tools
🪛 Ruff (0.11.9)
138-138: Loop control variable argument_function not used within loop body
(B007)
152-152: Trailing comma missing
Add trailing comma
(COM812)
🤖 Prompt for AI Agents
In .github/workflows/scripts/check_docstrings.py around lines 138 to 158, the
nested loops checking argument mismatches are inefficient and incorrect because
the inner condition does not use the outer loop variable, causing false
positives and O(n²) complexity. Replace the nested loops with set operations to
directly find arguments in the docstring not in the function and vice versa,
then append violations accordingly. This will simplify the logic, improve
performance, and eliminate loop-control-variable issues.
| - env: | ||
| CHANGED_FILES: ${{ steps.changed_files.outputs.all_changed_files }} | ||
| if: steps.changed_files.outputs.any_changed == 'true' || steps.changed_files.outputs.any_deleted == 'true' | ||
| name: Show changed files | ||
| run: | | ||
| echo "Unauthorized changes were made in the following files:" | ||
| for FILE in ${CHANGED_FILES}; do | ||
| echo "$FILE" | ||
| done | ||
| echo "To override this, apply the 'ignore-sensitive-files-pr' label" | ||
| exit 1 |
There was a problem hiding this comment.
Incorrect step ID referenced & missing outputs
steps.changed_files is undefined (the ID is changed-files). Additionally, all_changed_files_count is never written, so later checks always see an empty string.
- - env:
- CHANGED_FILES: ${{ steps.changed_files.outputs.all_changed_files }}
+ - env:
+ CHANGED_FILES: ${{ steps.changed-files.outputs.all_changed_files }}
...
- if: steps.changed_files.outputs.any_changed == 'true' || steps.changed_files.outputs.any_deleted == 'true'
+ if: steps.changed-files.outputs.any_changed == 'true' || steps.changed-files.outputs.any_deleted == 'true'In the changed-files step add:
echo "all_changed_files_count=$(echo \"$ALL_CHANGED_FILES\" | wc -w)" >> "$GITHUB_OUTPUT"so downstream steps can consume the count.
🧰 Tools
🪛 actionlint (1.7.7)
196-196: property "changed_files" is not defined in object type {changed-files: {conclusion: string; outcome: string; outputs: {string => string}}}
(expression)
197-197: property "changed_files" is not defined in object type {changed-files: {conclusion: string; outcome: string; outputs: {string => string}}}
(expression)
197-197: property "changed_files" is not defined in object type {changed-files: {conclusion: string; outcome: string; outputs: {string => string}}}
(expression)
🤖 Prompt for AI Agents
In .github/workflows/pull-request.yml around lines 195 to 205, the step ID
`changed_files` is incorrectly referenced and should be `changed-files`. Also,
the output `all_changed_files_count` is missing from the `changed-files` step.
Fix this by updating all references from `changed_files` to `changed-files` and
add the line `echo "all_changed_files_count=$(echo \"$ALL_CHANGED_FILES\" | wc
-w)" >> "$GITHUB_OUTPUT"` in the `changed-files` step to properly set the output
for downstream consumption.
| run: | | ||
| if [[ -n "$(git status --porcelain pubspec.lock)" ]]; then | ||
| echo "Uncommitted changes detected in pubspec.lock. Stashing just that file..." | ||
| git stash push -u -- "pubspec.lock" |
There was a problem hiding this comment.
🧹 Nitpick (assertive)
Use environment variables instead of untrusted expressions
github.head_ref is user-supplied. Pass it through env: and reference $PR_HEAD_REF inside the script to satisfy Action-lint’s security guidance.
🧰 Tools
🪛 actionlint (1.7.7)
60-60: "github.head_ref" is potentially untrusted. avoid using it directly in inline scripts. instead, pass it through an environment variable. see https://docs.github.com/en/actions/security-for-github-actions/security-guides/security-hardening-for-github-actions for more details
(expression)
🤖 Prompt for AI Agents
In .github/workflows/pull-request.yml around lines 60 to 63, the script uses the
user-supplied expression github.head_ref directly, which is a security risk. To
fix this, define an environment variable under env: (e.g., PR_HEAD_REF: ${{
github.head_ref }}) and then reference this variable inside the run script using
$PR_HEAD_REF instead of the direct expression. This change ensures safer
handling of user input in the workflow.
| run : | | ||
| # Get the base branch ref | ||
| BASE_SHA=$(git merge-base ${{ github.event.pull_request.base.sha }} ${{ github.event.pull_request.head.sha }}) | ||
|
|
||
| # Define sensitive files pattern | ||
| SENSITIVE_FILES=".flake8 .pydocstyle pyproject.toml .env* vitest.config.js src/App.tsx .github/** env.example .node-version .husky/** scripts/** src/style/** schema.graphql package.json package-lock.json tsconfig.json .gitignore .eslintrc.json .eslintignore .prettierrc .prettierignore vite.config.ts docker/docker-compose.prod.yaml docker/docker-compose.dev.yaml docker/Dockerfile.dev docker/Dockerfile.prod config/docker/setup/nginx.conf config/docker/setup/nginx.prod.conf CODEOWNERS LICENSE setup.ts .coderabbit.yaml CODE_OF_CONDUCT.md CODE_STYLE.md CONTRIBUTING.md DOCUMENTATION.md INSTALLATION.md ISSUE_GUIDELINES.md PR_GUIDELINES.md README.md *.pem *.key *.cert *.password *.secret *.credentials .nojekyll yarn.lock docs/docusaurus.config.ts docs/sidebar* CNAME" | ||
|
|
||
| # Check for changes in sensitive files |
There was a problem hiding this comment.
YAML syntax error – extra spaces before colon
run : | (note the space before :) is invalid YAML and will break the “Get Changed Unauthorized files” step.
- run : |
+ run: |📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| run : | | |
| # Get the base branch ref | |
| BASE_SHA=$(git merge-base ${{ github.event.pull_request.base.sha }} ${{ github.event.pull_request.head.sha }}) | |
| # Define sensitive files pattern | |
| SENSITIVE_FILES=".flake8 .pydocstyle pyproject.toml .env* vitest.config.js src/App.tsx .github/** env.example .node-version .husky/** scripts/** src/style/** schema.graphql package.json package-lock.json tsconfig.json .gitignore .eslintrc.json .eslintignore .prettierrc .prettierignore vite.config.ts docker/docker-compose.prod.yaml docker/docker-compose.dev.yaml docker/Dockerfile.dev docker/Dockerfile.prod config/docker/setup/nginx.conf config/docker/setup/nginx.prod.conf CODEOWNERS LICENSE setup.ts .coderabbit.yaml CODE_OF_CONDUCT.md CODE_STYLE.md CONTRIBUTING.md DOCUMENTATION.md INSTALLATION.md ISSUE_GUIDELINES.md PR_GUIDELINES.md README.md *.pem *.key *.cert *.password *.secret *.credentials .nojekyll yarn.lock docs/docusaurus.config.ts docs/sidebar* CNAME" | |
| # Check for changes in sensitive files | |
| run: | | |
| # Get the base branch ref | |
| BASE_SHA=$(git merge-base ${{ github.event.pull_request.base.sha }} ${{ github.event.pull_request.head.sha }}) | |
| # Define sensitive files pattern | |
| SENSITIVE_FILES=".flake8 .pydocstyle pyproject.toml .env* vitest.config.js src/App.tsx .github/** env.example .node-version .husky/** scripts/** src/style/** schema.graphql package.json package-lock.json tsconfig.json .gitignore .eslintrc.json .eslintignore .prettierrc .prettierignore vite.config.ts docker/docker-compose.prod.yaml docker/docker-compose.dev.yaml docker/Dockerfile.dev docker/Dockerfile.prod config/docker/setup/nginx.conf config/docker/setup/nginx.prod.conf CODEOWNERS LICENSE setup.ts .coderabbit.yaml CODE_OF_CONDUCT.md CODE_STYLE.md CONTRIBUTING.md DOCUMENTATION.md INSTALLATION.md ISSUE_GUIDELINES.md PR_GUIDELINES.md README.md *.pem *.key *.cert *.password *.secret *.credentials .nojekyll yarn.lock docs/docusaurus.config.ts docs/sidebar* CNAME" | |
| # Check for changes in sensitive files |
🧰 Tools
🪛 YAMLlint (1.37.1)
[error] 134-134: too many spaces before colon
(colons)
[error] 134-134: trailing spaces
(trailing-spaces)
🤖 Prompt for AI Agents
In .github/workflows/pull-request.yml around lines 134 to 141, remove the extra
space before the colon in the line "run : |" to correct the YAML syntax. Change
it to "run: |" so the step executes properly without YAML parsing errors.
| uses: actions/setup-python@v4 | ||
| with: | ||
| python-version: 3.11 | ||
|
|
There was a problem hiding this comment.
🛠️ Refactor suggestion
Out-of-date action version
actions/setup-python@v4 is flagged as too old; switch to @v5 for the Python-Compliance job to avoid deprecation warnings and future breakages.
- - name: Set up Python 3.11
- uses: actions/setup-python@v4
+ - name: Set up Python 3.11
+ uses: actions/setup-python@v5📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| uses: actions/setup-python@v4 | |
| with: | |
| python-version: 3.11 | |
| - name: Set up Python 3.11 | |
| uses: actions/setup-python@v5 | |
| with: | |
| python-version: 3.11 |
🧰 Tools
🪛 actionlint (1.7.7)
317-317: the runner of "actions/setup-python@v4" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
🤖 Prompt for AI Agents
In .github/workflows/pull-request.yml at lines 317 to 320, the GitHub action
uses an outdated version actions/setup-python@v4. Update the version to
actions/setup-python@v5 to prevent deprecation warnings and ensure compatibility
with future updates.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2831 +/- ##
==========================================
- Coverage 96.55% 95.58% -0.98%
==========================================
Files 189 186 -3
Lines 9994 9955 -39
==========================================
- Hits 9650 9515 -135
- Misses 344 440 +96 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Merged the latest develop code into main
Summary by CodeRabbit
New Features
Improvements
Bug Fixes
Chores
Removals