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

Update index.rst #1906

Merged
merged 1 commit into from Mar 20, 2020
Merged

Update index.rst #1906

merged 1 commit into from Mar 20, 2020

Conversation

HCSpin
Copy link
Contributor

@HCSpin HCSpin commented Mar 20, 2020

Requirements

  • Fill out this template to the extent possible so that this PR can be reviewed in a timely manner.
  • Replace the bracketed text below with your own.
  • All new code requires tests to ensure against regressions.

Description of the Change

[ Addition of a step in setting up the Excel wrapper]

Benefits

[ This action was required for me to get CoolProp to work on Excel 2019 for MacOSX. ]

Possible Drawbacks

[ Unknown ]

Verification Process

[ I tried locating the .dylib file elsewhere, providing the full path to it, but it only worked if I used the full path indicated in the rest of the installation guide. It did not work for me w/o the full path. It would be great if someone else could confirm on another mac.]

Applicable Issues

[ NA]

@CLAassistant
Copy link

CLAassistant commented Mar 20, 2020

CLA assistant check
All committers have signed the CLA.

@ibell
Copy link
Contributor

ibell commented Mar 20, 2020

Thank you!

@ibell ibell merged commit 6b51320 into CoolProp:master Mar 20, 2020
@ibell ibell added this to the v6.4.0 milestone May 9, 2020
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.

None yet

3 participants