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

added a task page to publish to Zenodo from GitLab #178

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

vuillaut
Copy link
Contributor

@vuillaut vuillaut commented Dec 18, 2024

Changes proposed in this pull request:

  • add a page to publish code from Zenodo to Github
  • renamed the zenodo_doi page into github_zenodo
  • modified the zenodo_doi page to include a solution using gitlab

Notes for reviewers:

Publishing code to Zenodo from GitHub is not the only way, and GitHub is not the main development platform in many communities. I added a page to indicate a possible alternative using GitLab.

Copy link

netlify bot commented Dec 19, 2024

Deploy Preview for everse-rsqkit-testing ready!

Name Link
🔨 Latest commit 2612ac9
🔍 Latest deploy log https://app.netlify.com/sites/everse-rsqkit-testing/deploys/6781480f6ac1c50008a60cae
😎 Deploy Preview https://deploy-preview-178--everse-rsqkit-testing.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@dgarijo dgarijo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @vuillaut
this task is great. I provided some suggestions to ensure consistency with other contents. Please review when you have the chance

_data/sidebars/main.yml Outdated Show resolved Hide resolved
pages/your_tasks/github_zenodo.md Outdated Show resolved Hide resolved
pages/your_tasks/gitlab_zenodo.md Outdated Show resolved Hide resolved
pages/your_tasks/gitlab_zenodo.md Outdated Show resolved Hide resolved
@vuillaut
Copy link
Contributor Author

Ok thank you for the feedback and comments @dgarijo
I will wait for #173 and then edit following your recommendations.

@vuillaut vuillaut requested a review from dgarijo January 10, 2025 15:57
@vuillaut
Copy link
Contributor Author

@dgarijo I have merged main and taken into account your feedback

@anenadic
Copy link
Contributor

anenadic commented Jan 10, 2025

@dgarijo I have merged main and taken into account your feedback

Thank you @vuillaut - the links to the two new pages are currently broken (I'll wait with the review in the case you are still working on this PR). You can preview your changes at https://deploy-preview-178--everse-rsqkit-testing.netlify.app/

@vuillaut
Copy link
Contributor Author

@dgarijo I have merged main and taken into account your feedback

Thank you @vuillaut - the links to the two new pages are currently broken (I'll wait with the review in the case you are still working on this PR). You can preview your changes at https://deploy-preview-178--everse-rsqkit-testing.netlify.app/

Sorry I was still fixing a few things.
I think it's good to review now.
Thanks !

@shoaibsufi shoaibsufi added the new content request New page request label Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new content request New page request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants