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 instructions: nuke VDI and OOD; add ARE #311

Closed
wants to merge 12 commits into from

Conversation

navidcy
Copy link
Collaborator

@navidcy navidcy commented Nov 20, 2022

This PR updates the instructions and removes all references to deprecated VDI system.

Closes #272
Closes #328

@micaeljtoliveira
Copy link
Contributor

What about ARE?

@navidcy
Copy link
Collaborator Author

navidcy commented Nov 21, 2022

ci/circleci: py36 — Your tests failed on CircleCI ?

@navidcy
Copy link
Collaborator Author

navidcy commented Nov 21, 2022

perhaps we wanna drop testing on python < 3.9?

README.md Outdated Show resolved Hide resolved
@angus-g angus-g changed the title Update instructions: nuke VDI, add OOD Update instructions: nuke VDI, add ARE Jan 25, 2023
README.md Outdated Show resolved Hide resolved
docs/source/getting_started.rst Outdated Show resolved Hide resolved
docs/source/getting_started.rst Outdated Show resolved Hide resolved
Copy link

codecov bot commented Feb 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 54.94%. Comparing base (a9f5fc9) to head (3730930).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #311   +/-   ##
=======================================
  Coverage   54.94%   54.94%           
=======================================
  Files          22       22           
  Lines        1811     1811           
=======================================
  Hits          995      995           
  Misses        816      816           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@navidcy navidcy requested a review from angus-g February 28, 2024 06:14
@navidcy
Copy link
Collaborator Author

navidcy commented Feb 28, 2024

I believe this is ready for review again ;)

@navidcy navidcy changed the title Update instructions: nuke VDI, add ARE Update instructions: nuke VDI and OOD; add ARE Feb 28, 2024
README.md Outdated Show resolved Hide resolved
docs/source/getting_started.rst Outdated Show resolved Hide resolved
docs/source/getting_started.rst Outdated Show resolved Hide resolved
docs/source/getting_started.rst Outdated Show resolved Hide resolved
@navidcy navidcy requested review from angus-g and aekiss February 28, 2024 06:20
@navidcy
Copy link
Collaborator Author

navidcy commented Feb 28, 2024

@aekiss does this PR resolve #328? Should we add more details for ARE?

navidcy and others added 4 commits February 28, 2024 08:20
Co-authored-by: Angus Gibson <[email protected]>
Co-authored-by: Angus Gibson <[email protected]>
@angus-g
Copy link
Collaborator

angus-g commented Feb 28, 2024

Thanks! Looks good from here. I'll let Andrew respond to your question above, but otherwise I'm happy.

@angus-g
Copy link
Collaborator

angus-g commented Jul 1, 2024

I've redirected the Cookbook documentation to that for the Recipes (#340), so I don't think this applies any more. This will probably fade out of relevance with the introduction of the Intake catalogue anyway.

@angus-g angus-g closed this Jul 1, 2024
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.

add ARE instructions to README Cookbook docs need attention
4 participants