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

debt: Abort deprecation #46

Closed
Ash258 opened this issue Jun 27, 2020 · 0 comments
Closed

debt: Abort deprecation #46

Ash258 opened this issue Jun 27, 2020 · 0 comments
Assignees
Labels
debt Code quality issues enhancement New feature or request meta released Changes already in main branch
Milestone

Comments

@Ash258
Copy link
Owner

Ash258 commented Jun 27, 2020

ScoopInstaller#3628, ScoopInstaller#3936, ScoopInstaller#4135, ScoopInstaller#4230

Future:

  • Investigate $PSScmdlet throw usage
  • Improve code in general
    • Code duplication should be possible to remove in case of handling the throws
@Ash258 Ash258 added enhancement New feature or request debt Code quality issues labels Jun 27, 2020
@Ash258 Ash258 added this to the 0.55 milestone Jun 27, 2020
@Ash258 Ash258 self-assigned this Jun 28, 2020
Ash258 added a commit that referenced this issue Jul 30, 2020
#46, #53

I did not expect that full deprecation will be that easy, so the original PR grow a bit and got really messy.
This is fraction of the previous PR with the easiest changes.

- Helper function
- Strict and safe `abort` replacement without additional side checks
@Ash258 Ash258 added the meta label Aug 7, 2020
@Ash258 Ash258 changed the title debt: Abort depracation debt: Abort deprecation Aug 16, 2020
@Ash258 Ash258 added the in-NEW Changes are already available in NEW branch. Configure SCOOP_BRANCH to NEW to beta-test changes label Aug 19, 2020
@Ash258 Ash258 added in-main Already merged to main branch and removed in-NEW Changes are already available in NEW branch. Configure SCOOP_BRANCH to NEW to beta-test changes labels Sep 5, 2020
@Ash258 Ash258 added in-NEW Changes are already available in NEW branch. Configure SCOOP_BRANCH to NEW to beta-test changes and removed in-main Already merged to main branch labels Oct 31, 2020
@Ash258 Ash258 closed this as completed in df733c2 Dec 7, 2020
@Ash258 Ash258 added in-main Already merged to main branch released Changes already in main branch and removed in-NEW Changes are already available in NEW branch. Configure SCOOP_BRANCH to NEW to beta-test changes in-main Already merged to main branch labels Dec 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
debt Code quality issues enhancement New feature or request meta released Changes already in main branch
Projects
None yet
Development

No branches or pull requests

1 participant