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

[3] Admin elevation #2601

Merged
merged 7 commits into from
Aug 28, 2024
Merged

Conversation

MyDrift-user
Copy link
Contributor

@MyDrift-user MyDrift-user commented Aug 21, 2024

Pull Request

Admin elevation

Type of Change

  • New feature
  • Refactoring
  • UI/UX improvement

Description

A way to launch winutil as admin with following upsides:

  • this needs no changes besides merging the PR
  • if script was ran from file it runs if from there, if it was ran using the one liner it uses the normal link. Also added the same logic to the windev to support the dev branch.

Testing

tested both wt & other, both worked for me

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no errors/warnings/merge conflicts.

@MyDrift-user MyDrift-user marked this pull request as draft August 21, 2024 20:00
@MyDrift-user MyDrift-user reopened this Aug 23, 2024
@MyDrift-user MyDrift-user marked this pull request as ready for review August 23, 2024 11:11
@ChrisTitusTech ChrisTitusTech force-pushed the main branch 2 times, most recently from 7805fe4 to cfb9edb Compare August 26, 2024 22:01
@ChrisTitusTech
Copy link
Owner

Much better than mine.

@ChrisTitusTech ChrisTitusTech merged commit cc157f6 into ChrisTitusTech:main Aug 28, 2024
1 check passed
@ChrisTitusTech ChrisTitusTech added the skip-changelog Skip Change Logs label Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog Skip Change Logs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants