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

(GH-13) Implement PRM install #42

Merged
merged 1 commit into from
Dec 9, 2021

Conversation

petergmurphy
Copy link
Contributor

@petergmurphy petergmurphy commented Dec 3, 2021

Overview of PR:

  • The install command has been created and all appropriate
    help information, arguments and parameters have been added to give it the same install
    functionality as that found on pct.

  • Added the config_processor.go with functions to appropriately process
    the prm config files after they're extracted from the tar.gz tool files.
    The ProcessConfig method is called from the genericised install package which is
    located in the pdkgo repository.

  • Added appropriate acceptance and unit testing along with test data and
    mocking modules to supplement the testing.

Resolves: #42

@petergmurphy petergmurphy force-pushed the gh-13/main/implement_prm_install branch 15 times, most recently from 6ccac5c to 4aac8d8 Compare December 9, 2021 10:36
@petergmurphy petergmurphy marked this pull request as ready for review December 9, 2021 10:42
@petergmurphy petergmurphy requested a review from a team as a code owner December 9, 2021 10:42
Copy link

@sanfrancrisko sanfrancrisko left a comment

Choose a reason for hiding this comment

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

Great stuff - just a few minor comments

cmd/install/install.go Outdated Show resolved Hide resolved
cmd/install/install.go Outdated Show resolved Hide resolved
acceptance/install/install_test.go Outdated Show resolved Hide resolved
@petergmurphy petergmurphy force-pushed the gh-13/main/implement_prm_install branch from 4aac8d8 to 9056023 Compare December 9, 2021 11:30
The install command has been created and all appropriate
help information, arguments and parameters have been added to give it the same install
functionality as that found on pct.

Added the `config_processor.go` with functions to appropriately process
the prm config files after they're extracted from the `tar.gz` tool files.
The `ProcessConfig` method is called from the genericised `install` package which is
located in the pdkgo repository.

Added appropriate acceptance and unit testing along with test data and
mocking modules to supplement the testing.
@petergmurphy petergmurphy force-pushed the gh-13/main/implement_prm_install branch from 9056023 to 921d602 Compare December 9, 2021 11:32
@sanfrancrisko sanfrancrisko linked an issue Dec 9, 2021 that may be closed by this pull request
@sanfrancrisko sanfrancrisko merged commit 3d4d347 into main Dec 9, 2021
@sanfrancrisko sanfrancrisko deleted the gh-13/main/implement_prm_install branch December 9, 2021 12:09
@petergmurphy petergmurphy added the enhancement New feature or request label May 31, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement prm install <pkg | url | git-url>
2 participants