Skip to content

Zenodo fixes - #281

Merged
andrewfullard merged 5 commits into
mainfrom
zenodo_fixes
Sep 8, 2025
Merged

Zenodo fixes#281
andrewfullard merged 5 commits into
mainfrom
zenodo_fixes

Conversation

@atharva-2001

@atharva-2001 atharva-2001 commented Aug 6, 2025

Copy link
Copy Markdown
Member

📝 Description

Type:
🪲 bugfix

  • Removes the bot token to GitHub token since GitHub token doesn't have sufficient permissions
  • Fixes the environment installation by reusing existing workflows.

📌 Resources

Examples, notebooks, and links to useful references.

🚦 Testing

How did you test these changes?

  • Testing pipeline
  • Other method (describe)
  • My changes can't be tested (explain why)

☑️ Checklist

  • I requested two reviewers for this pull request
  • I updated the documentation according to my changes
  • I built the documentation by applying the build_docs label

Note: If you are not allowed to perform any of these actions, ping (@) a contributor.

@andrewfullard

Copy link
Copy Markdown
Contributor

Looks like there are a lot of commented lines that should be removed

@andrewfullard andrewfullard left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please update the PR description

Comment thread .github/workflows/release.yml Outdated
@atharva-2001 atharva-2001 moved this from Todo to waiting for review in DevOps Aug 22, 2025
@atharva-2001 atharva-2001 self-assigned this Sep 5, 2025
@andrewfullard
andrewfullard merged commit 0adf265 into main Sep 8, 2025
11 checks passed
@github-project-automation github-project-automation Bot moved this from waiting for review to Done in DevOps Sep 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants