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 accessing remote data after dry-run #25

Merged
merged 5 commits into from
Feb 20, 2025

Conversation

jdangerx
Copy link
Member

@jdangerx jdangerx commented Feb 6, 2025

Overview

Closes #18 .

What problem does this address?

What did you change in this PR?

Testing

How did you make sure this worked? How can a reviewer verify this?

To-do list

Tasks

Preview Give feedback

Copy link

github-actions bot commented Feb 6, 2025

🆗 Pre-flight checks passed 😃

This pull request has been checked and contains no modified workflow files or spoofing.

Results of any additional workflows will appear here when they are done.

github-actions bot pushed a commit that referenced this pull request Feb 6, 2025
github-actions bot pushed a commit that referenced this pull request Feb 7, 2025
@jdangerx jdangerx force-pushed the update-accessing-remote-data branch from e475351 to a5d46ae Compare February 10, 2025 23:21
… section on trying things out and dealing with what happens; split EPA section into a constellation of small questions that walk through the framework.
@jdangerx jdangerx force-pushed the update-accessing-remote-data branch 2 times, most recently from 915058d to 0896ca3 Compare February 10, 2025 23:23
@jdangerx jdangerx force-pushed the update-accessing-remote-data branch from 0896ca3 to c3c55c5 Compare February 10, 2025 23:26
@jdangerx jdangerx force-pushed the update-accessing-remote-data branch 2 times, most recently from 44d08ad to 9723434 Compare February 20, 2025 15:07
@jdangerx
Copy link
Member Author

Asked about the CI failure in the Carpentries slack - there's an upstream issue that stops the GH-based builds from working, though this builds fine locally (see #30 for the build script). They're working on it. In the meantime I'm gonna merge this.

@jdangerx jdangerx merged commit b8b80f6 into main Feb 20, 2025
4 of 7 checks passed
@jdangerx jdangerx deleted the update-accessing-remote-data branch February 20, 2025 15:27
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.

Episode 3 fixes from first run-through
1 participant