Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue #27 capitalize the brand variable for the Xiaomi brand #50

Closed
wants to merge 1 commit into from
Closed

Issue #27 capitalize the brand variable for the Xiaomi brand #50

wants to merge 1 commit into from

Conversation

prathamdby
Copy link


Summary:

This pull request resolves Issue #27 by ensuring the correct capitalization of the brand variable for Xiaomi devices. Previously, the $BRAND variable incorrectly held the value "xiaomi" instead of the standard "Xiaomi". By adding a line in rom_vars.sh, this issue is rectified, aligning with the expected brand representation.

Changes Made:

  • Added a line in rom_vars.sh to capitalize the brand variable for Xiaomi devices.

Purpose:

The primary purpose of this pull request is to fix the capitalization inconsistency in the brand variable for Xiaomi devices. By ensuring that the $BRAND variable holds the correct capitalization, this enhancement improves code clarity and maintains consistency across the codebase, contributing to a better user experience.

How to Test:

  1. Pull the latest changes from this branch.
  2. Verify that the $BRAND variable is correctly capitalized for Xiaomi devices.
  3. Ensure that the fix does not introduce any regressions or unexpected behavior.

Related Issues:

Additional Notes:

This pull request adheres to the project's coding standards and has been tested to ensure proper functionality. Feedback and suggestions for further improvement are welcome.


@prathamdby prathamdby marked this pull request as draft March 17, 2024 13:23
@prathamdby prathamdby marked this pull request as ready for review March 17, 2024 13:24
@prathamdby prathamdby closed this Apr 13, 2024
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.

Minor derp in dummy_dt function
1 participant