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

Pave path v4 #1

Closed
wants to merge 31 commits into from
Closed

Pave path v4 #1

wants to merge 31 commits into from

Conversation

validcube
Copy link

Description

Fixes #(issue)

Type of change

❌ General change (non-breaking change that doesn't fit the below categories like copyediting)
❌ Bug fix (non-breaking change which fixes an issue)
❌ New feature (non-breaking change which adds functionality)
❌ Breaking change (fix or feature that would cause existing functionality to not work as expected)

Choose a reason for hiding this comment

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

Copilot reviewed 68 out of 82 changed files in this pull request and generated no suggestions.

Files not reviewed (14)
  • gradle.properties: Language not supported
  • gradle/wrapper/gradle-wrapper.properties: Language not supported
  • lawnchair/res/layout-v28/task_desktop.xml: Language not supported
  • lawnchair/res/values-fr-rFR/strings.xml: Language not supported
  • lawnchair/res/values-hu-rHU/strings.xml: Language not supported
  • lawnchair/res/values-in-rID/strings.xml: Language not supported
  • flags/src/com/android/window/flags2/CustomFeatureFlags.java: Evaluated as low risk
  • .github/workflows/crowdin_download.yml: Evaluated as low risk
  • flags/src/com/android/window/flags2/Flags.java: Evaluated as low risk
  • flags/src/com/android/window/flags2/FeatureFlagsImpl.java: Evaluated as low risk
  • flags/src/com/android/window/flags2/FeatureFlags.java: Evaluated as low risk
  • SECURITY.md: Evaluated as low risk
  • flags/src/com/android/window/flags2/FakeFeatureFlagsImpl.java: Evaluated as low risk
  • flags/src/com/android/launcher3/FeatureFlagsImpl.java: Evaluated as low risk
Comments skipped due to low confidence (1)

README.md:20

  • [nitpick] The spaces around the em dashes were removed. For consistency, consider adding spaces around the em dashes: 'Taking Launcher3 — Android’s default home app — as a starting point'.
Lawnchair is a free, open-source home app for Android. Taking Launcher3—Android’s default home app—as a starting point, it ports Pixel Launcher features and introduces rich customization options.
@validcube validcube requested a review from Copilot December 1, 2024 12:01

Choose a reason for hiding this comment

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

Copilot reviewed 11 out of 13 changed files in this pull request and generated no suggestions.

Files not reviewed (2)
  • .github/workflows/crowdin.yml: Evaluated as low risk
  • CONTRIBUTING.md: Evaluated as low risk
@validcube validcube requested a review from Copilot December 7, 2024 12:34

Choose a reason for hiding this comment

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

Copilot reviewed 14 out of 14 changed files in this pull request and generated no suggestions.

Signed-off-by: Pun Butrach <[email protected]>
@validcube validcube requested a review from Copilot December 7, 2024 12:48
Copy link

@Copilot 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.

Copilot reviewed 11 out of 14 changed files in this pull request and generated 1 suggestion.

Files not reviewed (3)
  • .github/workflows/crowdin.yml: Evaluated as low risk
  • CONTRIBUTING.md: Evaluated as low risk
  • compatLib/README.md: Evaluated as low risk

@validcube validcube requested a review from Copilot December 7, 2024 13:03
Copy link

@Copilot 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.

Copilot reviewed 11 out of 14 changed files in this pull request and generated 3 suggestions.

Files not reviewed (3)
  • compatLib/README.md: Evaluated as low risk
  • .github/workflows/crowdin_upload.yml: Evaluated as low risk
  • .github/workflows/crowdin.yml: Evaluated as low risk
Comments skipped due to low confidence (1)

SECURITY.md:37

  • The word 'endeavour' should be 'endeavor' to match American English spelling used elsewhere in the document.
We'll endeavour to respond quickly, and will keep you updated throughout the process.

README.md Outdated

### Development builds

Interested in keeping yourself up-to-date with every Lawnchair development? Try our development build, these build offered the latest features and bug fixes at a cost of being slower and *might* introduce new bugs that have never been seen in stable. If you'd like to join in the fun, don't forget to make backup(s) and proceed with cautions.
Copy link
Preview

Copilot AI Dec 7, 2024

Choose a reason for hiding this comment

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

The phrase 'these build offered' should be 'these builds offer'.

Suggested change
Interested in keeping yourself up-to-date with every Lawnchair development? Try our development build, these build offered the latest features and bug fixes at a cost of being slower and *might* introduce new bugs that have never been seen in stable. If you'd like to join in the fun, don't forget to make backup(s) and proceed with cautions.
Interested in keeping yourself up-to-date with every Lawnchair development? Try our development build, these builds offer the latest features and bug fixes at a cost of being slower and *might* introduce new bugs that have never been seen in stable. If you'd like to join in the fun, don't forget to make backup(s) and proceed with cautions.

Copilot is powered by AI, so mistakes are possible. Review output carefully before use.

Positive Feedback
Negative Feedback

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

Please select one or more of the options
This directory contains all of the required SystemUI module like Common as helper for other SystemUI modules like Unfold for handling devices with hinges.
Copy link
Preview

Copilot AI Dec 7, 2024

Choose a reason for hiding this comment

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

The phrase 'all of the required SystemUI module like Common as helper for other SystemUI modules like Unfold for handling devices with hinges' is grammatically incorrect. It should be 'all of the required SystemUI modules, such as Common, which serves as a helper for other SystemUI modules like Unfold for handling devices with hinges.'

Suggested change
This directory contains all of the required SystemUI module like Common as helper for other SystemUI modules like Unfold for handling devices with hinges.
This directory contains all of the required SystemUI modules, such as Common, which serves as a helper for other SystemUI modules like Unfold for handling devices with hinges.

Copilot is powered by AI, so mistakes are possible. Review output carefully before use.

Positive Feedback
Negative Feedback

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

Please select one or more of the options
@@ -33,7 +33,7 @@ For translation, visit [Lawnchair on Crowdin](https://lawnchair.crowdin.com) and
### 🧑‍💻 Code

> [!NOTE]
> For Lawnchair 9 to 13, see the branches with the `9-` to `13-` prefixes respectively.
> For Lawnchair 9 to 14, see the branches with the `9-` to `14-` prefixes respectively.
Copy link
Preview

Copilot AI Dec 7, 2024

Choose a reason for hiding this comment

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

The sentence should be 'For Lawnchair 9 to 14, see the branches with the 9- to 14- prefixes, respectively.'

Suggested change
> For Lawnchair 9 to 14, see the branches with the `9-` to `14-` prefixes respectively.
> For Lawnchair 9 to 14, see the branches with the `9-` to `14-` prefixes, respectively.

Copilot is powered by AI, so mistakes are possible. Review output carefully before use.

Positive Feedback
Negative Feedback

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

Please select one or more of the options
@validcube validcube requested a review from Copilot December 7, 2024 13:07

Choose a reason for hiding this comment

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

Copilot reviewed 11 out of 14 changed files in this pull request and generated no suggestions.

Files not reviewed (3)
  • .github/workflows/crowdin.yml: Evaluated as low risk
  • .github/workflows/crowdin_download.yml: Evaluated as low risk
  • .github/workflows/crowdin_upload.yml: Evaluated as low risk
@validcube validcube requested a review from Copilot December 12, 2024 13:30
Copy link

@Copilot 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.

Copilot reviewed 11 out of 14 changed files in this pull request and generated 1 suggestion.

Files not reviewed (3)
  • compatLib/README.md: Evaluated as low risk
  • .github/workflows/crowdin_download.yml: Evaluated as low risk
  • .github/workflows/crowdin_upload.yml: Evaluated as low risk

@validcube validcube requested a review from Copilot December 12, 2024 14:29
Copy link

@Copilot 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.

Copilot reviewed 11 out of 14 changed files in this pull request and generated 1 suggestion.

Files not reviewed (3)
  • compatLib/README.md: Evaluated as low risk
  • .github/workflows/crowdin.yml: Evaluated as low risk
  • .github/workflows/crowdin_upload.yml: Evaluated as low risk

Co-authored-by: Copilot <[email protected]>
Signed-off-by: Pun Butrach <[email protected]>
@validcube validcube requested a review from Copilot December 13, 2024 01:59
Copy link

@Copilot 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.

Copilot reviewed 11 out of 14 changed files in this pull request and generated 1 comment.

Files not reviewed (3)
  • compatLib/README.md: Evaluated as low risk
  • .github/workflows/crowdin.yml: Evaluated as low risk
  • .github/workflows/crowdin_download.yml: Evaluated as low risk
Comments suppressed due to low confidence (2)

CONTRIBUTING.md:31

  • The phrase 'as listed in this [this summary table]' should be 'as listed in this [summary table]'.
When using quotation marks, insert the symbols specific to the target language, as listed in this [this summary table](https://en.wikipedia.org/wiki/Quotation_mark#Summary_table);

SECURITY.md:31

  • [nitpick] The word 'endeavor' should be 'endeavour' to maintain consistency with British English spelling used elsewhere in the document.
We'll endeavor to respond quickly, and will keep you updated throughout the process.

- QuickSwitch support
- Global Search support
- Icons, fonts, and colors customization
- **Material You Theming:** Adapts to your wallpaper and system theme.
Copy link
Preview

Copilot AI Dec 13, 2024

Choose a reason for hiding this comment

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

The phrase 'Material You Theming' should be 'Material You theming' for consistency with capitalization rules.

Suggested change
- **Material You Theming:** Adapts to your wallpaper and system theme.
- **Material You theming:** Adapts to your wallpaper and system theme.

Copilot is powered by AI, so mistakes are possible. Review output carefully before use.

Positive Feedback
Negative Feedback

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

Please select one or more of the options
@validcube validcube closed this Feb 5, 2025
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.

1 participant