Skip to content

Conversation

@mir-huzaif
Copy link
Contributor

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds a new FFI function binding for MigrateToSystemWideActivation to the LexActivator Ruby module, enabling migration of activations to system-wide scope.

Changes:

  • Added MigrateToSystemWideActivation function binding with proper FFI signature and documentation

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +600 to +601
# @method MigrateToSystemWideActivation(oldPermissionFlag)
# @param [Integer] oldPermissionFlag
Copy link

Copilot AI Jan 14, 2026

Choose a reason for hiding this comment

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

The parameter name "oldPermissionFlag" uses camelCase, which is inconsistent with the naming convention used throughout this file. Other functions use snake_case for parameter names (e.g., "file_path", "license_key", "app_version", "trial_extension_length"). Consider renaming to "old_permission_flag" for consistency.

Suggested change
# @method MigrateToSystemWideActivation(oldPermissionFlag)
# @param [Integer] oldPermissionFlag
# @method MigrateToSystemWideActivation(old_permission_flag)
# @param [Integer] old_permission_flag

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants