conda-forge / qcportal-feedstock

A conda-smithy repository for qcportal.
BSD 3-Clause "New" or "Revised" License
0 stars 6 forks source link

Update pydantic pin #32

Closed mikemhenry closed 1 year ago

mikemhenry commented 1 year ago

Checklist

conda-forge-webservices[bot] commented 1 year ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

mikemhenry commented 1 year ago

Resolves #31

mikemhenry commented 1 year ago

@conda-forge-admin, please rerender

github-actions[bot] commented 1 year ago

Hi! This is the friendly automated conda-forge-webservice.

I tried to rerender for you, but it looks like there was nothing to do.

This message was generated by GitHub actions workflow run https://github.com/conda-forge/qcportal-feedstock/actions/runs/5557432173.

mikemhenry commented 1 year ago

I'm not sure when this was switch from a warning to a error:

2023-07-14T19:06:45.7626957Z     def check_dict_or_set_indexers(key) -> None:
2023-07-14T19:06:45.7627176Z         """
2023-07-14T19:06:45.7627406Z         Check if the indexer is or contains a dict or set, which is no longer allowed.
2023-07-14T19:06:45.7627642Z         """
2023-07-14T19:06:45.7627822Z         if (
2023-07-14T19:06:45.7628029Z             isinstance(key, set)
2023-07-14T19:06:45.7628250Z             or isinstance(key, tuple)
2023-07-14T19:06:45.7628485Z             and any(isinstance(x, set) for x in key)
2023-07-14T19:06:45.7628704Z         ):
2023-07-14T19:06:45.7628896Z >           raise TypeError(
2023-07-14T19:06:45.7629137Z                 "Passing a set as an indexer is not supported. Use a list instead."
2023-07-14T19:06:45.7629420Z             )
2023-07-14T19:06:45.7629664Z E           TypeError: Passing a set as an indexer is not supported. Use a list instead.

But it is coming from pandas. Skimming the change log, I think I found where the switch happened, but I could be wrong.

bennybp commented 1 year ago

@conda-forge-admin, please rerender

bennybp commented 1 year ago

Sorry for the delay. This looks reasonable to me.

For the next release, we are decommissioning/archiving this feedstock in favor of building all packages in the qcfractal-feedstock. But hopefully this build will stick around for a while.