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 sync.yml #374

Merged
merged 3 commits into from
Jan 11, 2022
Merged

Update sync.yml #374

merged 3 commits into from
Jan 11, 2022

Conversation

arpanda
Copy link
Contributor

@arpanda arpanda commented Dec 24, 2021

Purpose/implementation Section

Add CNVpytor_course

What changes are being implemented in this Pull Request?

What was your approach?

What GitHub issue does your pull request address?

Tell potential reviewers what kind of feedback you are soliciting.

New Content Checklist

@carriewright11 carriewright11 self-assigned this Jan 6, 2022
@cansavvy cansavvy mentioned this pull request Jan 7, 2022
@cansavvy
Copy link
Collaborator

cansavvy commented Jan 7, 2022

Ah this is failing because its coming from a forked repository and doesn't have the necessary permissions. Perhaps this is a reason for me to work on #330

@cansavvy
Copy link
Collaborator

Some of the problems are fixed. I'm going to merge this though since we know it has changes we want.

@cansavvy cansavvy merged commit ebc6079 into jhudsl:main Jan 11, 2022
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.

3 participants