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

REL: 1.8.7 #3639

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

REL: 1.8.7 #3639

wants to merge 5 commits into from

Conversation

effigies
Copy link
Member

@effigies effigies commented Mar 20, 2024

Preparation for the 1.8.7 release. It's just been a while and we have over 184 commits since the last release. This is not urgent.

The following users have made their first contributions:

Please let me know if you would like to be credited in releases, and if so what name, ORCID and affiliation you would like. See https://github.com/nipy/nipype/blob/master/.zenodo.json for examples. If you have already added yourself, no need to respond.

Quick issues/PRs:

  • ... Please let me know if you want to try to merge something in before the release.

Release checklist

  • Merge pending PRs
  • Update changelog
  • Update .mailmap
  • Update .zenodo.json
  • Set release number in nipype/info.py
  • Update doc/interfaces.rst with previous releases

Copy link

codecov bot commented Mar 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.90%. Comparing base (d52a62d) to head (391c238).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3639   +/-   ##
=======================================
  Coverage   70.90%   70.90%           
=======================================
  Files        1277     1277           
  Lines       59477    59477           
  Branches     9819     9819           
=======================================
  Hits        42172    42172           
  Misses      16134    16134           
  Partials     1171     1171           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@yibeichan
Copy link
Contributor

thank you! but my PR was a very minor one...next time when I make more contributions I'll add myself

@effigies
Copy link
Member Author

@djarecka Can you create a new TUTORIAL_ACCESS_TOKEN? I believe it's a PAT on https://github.com/miykael/nipype_tutorial. I do not have permissions there.

@djarecka
Copy link
Collaborator

@djarecka Can you create a new TUTORIAL_ACCESS_TOKEN? I believe it's a PAT on https://github.com/miykael/nipype_tutorial. I do not have permissions there.

TUTORIAL_ACCESS_TOKEN is nipype secret, I don't have admin access on nipype tutorial, but can try to update out secret with my PAT

@effigies
Copy link
Member Author

effigies commented Mar 21, 2024

Token seems to work, but it looks like something went wrong with the action itself: https://github.com/miykael/nipype_tutorial/actions/runs/8376744890

@djarecka
Copy link
Collaborator

Token seems to work, but it looks like something went wrong with the action itself: https://github.com/miykael/nipype_tutorial/actions/runs/8376744890

yes, looks like there is problem with building

@effigies
Copy link
Member Author

@mgxd I think you last fixed things related to DLC in miykael/nipype_tutorial#182. Any ideas?

@djarecka
Copy link
Collaborator

slightly more info here: https://github.com/miykael/nipype_tutorial/actions/runs/8377012805/job/22938063788

Can also take a look later today

@mgxd
Copy link
Member

mgxd commented Mar 21, 2024

Looks like the DLC action we were using has been orphaned - there's a (somewhat) maintained fork here https://github.com/jpribyl/action-docker-layer-caching

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.

None yet

4 participants