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

Willaguiar/main #483

Closed
wants to merge 4 commits into from
Closed

Willaguiar/main #483

wants to merge 4 commits into from

Conversation

julia-neme
Copy link
Collaborator

No description provided.

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@julia-neme
Copy link
Collaborator Author

I tried to push this to #402 but I don't know why its created a new pull request. Sorry

@navidcy
Copy link
Collaborator

navidcy commented Feb 5, 2025

#402 is a PR from @willaguiar's fork: https://github.com/willaguiar/cosima-recipes/tree/main
you need to push directly on that fork to modify the PR!

seems you made a new branch on your fork and you named it willaguiar/main and then opened a PR from there... that's why the PR was opened to merge whatever you had in your branch to the repo's main.

@navidcy navidcy closed this Feb 5, 2025
@navidcy
Copy link
Collaborator

navidcy commented Feb 5, 2025

(I closed the PR. Re-open if you like! Your branch should still be there!)

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.

2 participants