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.

Sign up for GitHub

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

Jump to bottom

Refactor NpmLocation class #2038

Merged
merged 8 commits into from
Dec 18, 2023
Merged

Refactor NpmLocation class #2038

merged 8 commits into from
Dec 18, 2023

Commits on Dec 13, 2023

  1. Refactor NPM location class

    @FrederikBolding
    FrederikBolding committed Dec 13, 2023
    Configuration menu
    Copy the full SHA
    25ffa87 View commit details
    Browse the repository at this point in the history

Commits on Dec 14, 2023

  1. Update coverage

    @FrederikBolding
    FrederikBolding committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    722a525 View commit details
    Browse the repository at this point in the history

  2. More refactoring

    @FrederikBolding
    FrederikBolding committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    9e1c480 View commit details
    Browse the repository at this point in the history

  3. Improve testing

    @FrederikBolding
    FrederikBolding committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    1874169 View commit details
    Browse the repository at this point in the history

Commits on Dec 15, 2023

  1. Address PR comments

    @FrederikBolding
    FrederikBolding committed Dec 15, 2023
    Configuration menu
    Copy the full SHA
    65c7b45 View commit details
    Browse the repository at this point in the history

  2. Apply suggestions from code review 

    Co-authored-by: Maarten Zuidhoorn <[email protected]>
    @FrederikBolding @Mrtenz
    FrederikBolding and Mrtenz authored Dec 15, 2023
    Configuration menu
    Copy the full SHA
    1be9bad View commit details
    Browse the repository at this point in the history

Commits on Dec 18, 2023

  1. Merge branch 'main' into fb/npm-refactor

    @FrederikBolding
    FrederikBolding authored Dec 18, 2023
    Configuration menu
    Copy the full SHA
    54b60cf View commit details
    Browse the repository at this point in the history

  2. Add test

    @FrederikBolding
    FrederikBolding committed Dec 18, 2023
    Configuration menu
    Copy the full SHA
    7af8763 View commit details
    Browse the repository at this point in the history