redhat-developer / alizer

⛔️ DEPRECATED - Application analyzer toolkit, use https://github.com/devfile/alizer instead
Apache License 2.0
12 stars 17 forks source link

Add min max cli args #234

Closed thepetk closed 1 year ago

thepetk commented 1 year ago

What does this PR do?

As part of #159 we are introducing the two first cli args in order to support specific versions of devfiles. Those two are min-version and max-version.

If this argument is passed in the cli we are updating the URL for the download of the devfiles from the registry (following this link). This way we are filtering the list of devfiles we are getting from the registry.

Some edge cases:

A slight addition is also made to the model.DevfileType in order to include information for versions. For backwards compatibility reasons we are no longer using SelectDevFilesFromRegistry but MatchDevfiles.

A new model.DevfileFilter is created in order to include all future filters inside the devfile API and ensure scalability.

Updates are made inside the documentation of the project to include all changes introduced and test cases regarding the URL update are added.

Which issue(s) does this PR fix

fixes #226

PR acceptance criteria

Testing and documentation do not need to be complete in order for this PR to be approved. We just need to ensure tracking issues are opened.

How to test changes / Special notes to the reviewer

thepetk commented 1 year ago

closing in favor of https://github.com/devfile/alizer/pull/5