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

rdkit-pypi replaced by rdkit on pypi #108

Merged
merged 2 commits into from
Feb 16, 2023
Merged

Conversation

IAlibay
Copy link
Contributor

@IAlibay IAlibay commented Feb 16, 2023

Small thing, I believe rdkit now replaces rdkit-pypi for new releases.

@cbouy
Copy link
Member

cbouy commented Feb 16, 2023

Right, thanks Irfan!

@IAlibay
Copy link
Contributor Author

IAlibay commented Feb 16, 2023

might be using it as an excuse to cycle CI a little bit ;)

@cbouy
Copy link
Member

cbouy commented Feb 16, 2023

Looks like the CI pipeline needs some tweaking: conda-incubator/setup-miniconda#274 🤷‍♂️

@IAlibay
Copy link
Contributor Author

IAlibay commented Feb 16, 2023

I forgot about that issue 🤣 fix incoming

@codecov
Copy link

codecov bot commented Feb 16, 2023

Codecov Report

Merging #108 (53bac75) into master (76249b3) will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #108      +/-   ##
==========================================
+ Coverage   95.71%   95.74%   +0.03%     
==========================================
  Files           9       10       +1     
  Lines        1073     1081       +8     
==========================================
+ Hits         1027     1035       +8     
  Misses         46       46              
Impacted Files Coverage Δ
prolif/__init__.py 100.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@cbouy cbouy merged commit d4939d5 into chemosim-lab:master Feb 16, 2023
@cbouy
Copy link
Member

cbouy commented Feb 16, 2023

Thanks Irfan!

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.

2 participants