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

Install on OSX #1257

Closed
wants to merge 5 commits into from
Closed

Install on OSX #1257

wants to merge 5 commits into from

Conversation

valeriupredoi
Copy link
Contributor

No description provided.

@valeriupredoi
Copy link
Contributor Author

ok @bouweandela updated the instructions and added warning that these are not continuously tested via CircleCI - let's get this in 🍺

@valeriupredoi
Copy link
Contributor Author

OK @bouweandela don't take your guns to town, I had to rerun the CI tests manually here, no more fails, it was going bananas yesterday tho 😁

Copy link
Member

@bouweandela bouweandela left a comment

Choose a reason for hiding this comment

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

It looks like all the steps are the same as for linux, except for the conda download URL and the warnings about not tested and gcc. Could you remove all the duplicate instructions and just mention what's different?

@mattiarighi mattiarighi changed the base branch from version2_development to master January 3, 2020 18:00
@bouweandela
Copy link
Member

@valeriupredoi This looks a bit outdated. is it still relevant?

@valeriupredoi
Copy link
Contributor Author

this an ancient branch that is most probably not useful to you but am tagging you anyhoo @bvreede - I am going to close it since it's too old and I am fairly against a separate env file anyway (me from 2021 does not agree with me from back in 2019) 😁

@bouweandela bouweandela deleted the version2_development_installOSX branch June 25, 2021 10:29
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.

2 participants