Skip to content
This repository has been archived by the owner on Nov 20, 2023. It is now read-only.

feat: add Shizuku installation support #83

Open
wants to merge 7 commits into
base: dev
Choose a base branch
from
Open

Conversation

Aunali321
Copy link
Contributor

@Aunali321 Aunali321 commented Aug 14, 2023

Add support for Shizuku installer

@Aunali321 Aunali321 changed the title feat: add Shizuku installation support feat: add Shizuku installation support (WIP) Aug 14, 2023
@Aunali321 Aunali321 marked this pull request as ready for review August 14, 2023 19:45
@Aunali321 Aunali321 changed the title feat: add Shizuku installation support (WIP) feat: add Shizuku installation support Aug 14, 2023
app/build.gradle.kts Outdated Show resolved Hide resolved
app/src/main/res/xml/provider_paths.xml Show resolved Hide resolved
app/src/main/AndroidManifest.xml Outdated Show resolved Hide resolved
app/src/main/res/values/strings.xml Outdated Show resolved Hide resolved
Aunali321 and others added 4 commits August 23, 2023 01:13
# Conflicts:
#	app/src/main/java/app/revanced/manager/ui/screen/InstalledAppsScreen.kt
#	app/src/main/java/app/revanced/manager/ui/viewmodel/InstallerViewModel.kt
})

val installer by prefs.defaultInstaller.getAsState()
GroupHeader(stringResource(R.string.installer))
Copy link
Member

Choose a reason for hiding this comment

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

The formatting in this file is weird

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Default android studio formatting

Copy link
Member

@Axelen123 Axelen123 Aug 24, 2023

Choose a reason for hiding this comment

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

Keep the custom line breaks next time. The rest of the codebase also has them.

Here is a version of the file with the line breaks restored: https://gist.github.com/Axelen123/834748677989c88531cc01e7194cab2e

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants