Skip to content

Fixes for mulled search#13532

Merged
mvdbeek merged 3 commits intogalaxyproject:devfrom
nsoranzo:fix_github_API_rate_limit
Mar 14, 2022
Merged

Fixes for mulled search#13532
mvdbeek merged 3 commits intogalaxyproject:devfrom
nsoranzo:fix_github_API_rate_limit

Conversation

@nsoranzo
Copy link
Member

  • Check GitHub API rate limit in GitHubSearch. Deprecate get_json() search.
    Follow-up on Add retries to test_github_search #13509 .
  • Fix CondaSearch to use conda from PATH and a new CondaContext.exec_search() method
    The conda package on PyPI is severely outdated and unmaintained, and the output of run_command("search", ...) did not parse correctly.
  • Rework best_search_result() and CondaInDockerContext to use CondaContext.exec_search().
  • Fix broken test_conda_search unit test.

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

Rework `best_search_result()` and `CondaInDockerContext` to use it.
…rch()`

The `conda` package on PyPI is severely outdated and unmaintained, and
the output of `run_command("search", ...)` did not parse correctly.

Fix broken `test_conda_search` unit test.
@mvdbeek
Copy link
Member

mvdbeek commented Mar 14, 2022

Thank you @nsoranzo!

@nsoranzo nsoranzo deleted the fix_github_API_rate_limit branch March 14, 2022 09:12
nsoranzo added a commit to nsoranzo/planemo that referenced this pull request Apr 1, 2022
In galaxyproject/galaxy#13532 I've removed
the `channels_override` parameter from the `best_search_result()`
method in galaxy-tool-util.
mvdbeek added a commit that referenced this pull request Apr 4, 2022
…364a20835acf16ccdffea3018a58023e2d

[21.05] Partially backport #13532 to fix `test_github_search` mulled unit test
nsoranzo added a commit to nsoranzo/planemo that referenced this pull request Apr 19, 2022
In galaxyproject/galaxy#13532 I've removed
the `channels_override` parameter from the `best_search_result()`
method in galaxy-tool-util.
nsoranzo added a commit to nsoranzo/planemo that referenced this pull request Apr 19, 2022
In galaxyproject/galaxy#13532 I've removed
the `channels_override` parameter from the `best_search_result()`
method in galaxy-tool-util.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants