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

Step 0 does not affect the end result #159

Merged
merged 1 commit into from
Dec 14, 2023

Conversation

asmacdo
Copy link
Collaborator

@asmacdo asmacdo commented Nov 20, 2023

It is helpful to point out that these steps do not make changes to their babs project. I pointlessly got into the habit of doing this every time.

It is helpful to point out that these steps do not make changes
to their babs project. I pointlessly got into the habit of doing
this every time.
@zhao-cy
Copy link
Collaborator

zhao-cy commented Nov 20, 2023

Oh no sorry about that. Sounds good! Let me know if you think it's done here / it's ready to be merged? If so, I'll merge it into main. Thanks!

@asmacdo
Copy link
Collaborator Author

asmacdo commented Nov 20, 2023

@zhao-cy thanks, I think it's good to go!

@asmacdo asmacdo merged commit 727f312 into PennLINC:main Dec 14, 2023
2 checks passed
@zhao-cy
Copy link
Collaborator

zhao-cy commented Dec 14, 2023

Hey @asmacdo ! Thanks a lot! Sorry for waiting for a long time.. There was an error in circle ci when merging but I reran circle ci (pytest part) and it passes now. I guess there were some temporary issues (e.g., connection to OSF?) that caused the error.

I'm graduating and leaving the lab in the next several days. If you have any new PRs, please ask in the BABS slack channel to see who will be the best person to review. Thanks a lot for all of you've done!!!

@zhao-cy zhao-cy added the documentation Improvements or additions to documentation label Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants