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

handle pkgs with no timestamps #12

Merged
merged 7 commits into from
Apr 12, 2021

Conversation

jayfurmanek
Copy link
Contributor

Checklist before submitting

Description

Some packages in defaults don't have timestamps!
For example: cookies-2.2.1-py36_0.tar.bz2

This checks for a timestamp before doing checks with it.

Review process to land

  1. All tests and other checks must succeed.
  2. At least one maintainer must review and approve.
  3. If any maintainer requests changes, they must be addressed.

open_ce/conda_utils.py Outdated Show resolved Hide resolved
@@ -41,6 +41,20 @@ def conda_search_result():
subdir : linux-ppc64le
url : https://repo.anaconda.com/pkgs/main/linux-ppc64le/
md5 : 4691146ff587f371f83f0e7bab93b63b
dependencies:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added a no timestamp version here too, though its not really a problem when doing env file validation at the moment.

Copy link
Member

@bethune-bryant bethune-bryant left a comment

Choose a reason for hiding this comment

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

LGTM
Looks like it will prefer results that have timestamps, if there's ever one that doesn't and one that does, which is probably good.

@jayfurmanek
Copy link
Contributor Author

yeah - at this point we''re assuming only mistaken or old out of date packages have missing timestamps.

@jayfurmanek jayfurmanek merged commit 51f7dde into open-ce:main Apr 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants