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

CompatHelper: bump compat for MRIReco to 0.8, (keep existing compat) #186

Merged

Conversation

github-actions[bot]
Copy link
Contributor

This pull request changes the compat entry for the MRIReco package from 0.6, 0.7 to 0.6, 0.7, 0.8.
This keeps the compat entries for earlier versions.

Note: I have not tested your package with this new compat entry.
It is your responsibility to make sure that your package tests pass before you merge this pull request.

@cncastillo
Copy link
Member

@beorostica could you check why the CI for this not running automatically?

@beorostica
Copy link
Contributor

beorostica commented Oct 24, 2023

After reading some discussions, it's not possible to trigger pull requests this way, refer to this and this. Summarizing, it is not allowed to trigger events from a task that uses the GITHUB_TOKEN, this is for avoiding to create recursive workflows.

Apparently, there are some workarounds (shown in the previous links) but we need to investigate further in order to implement them,

By the time being, the solution is to manually perform the merge action from 'compat' branches.

@cncastillo
Copy link
Member

I think the problem was simpler, the branch was out of date, when I pressed "update branch" it started running the CI.

@beorostica
Copy link
Contributor

It is a good idea to be aware of the behavior when CompatHelper detects a new update. I cannot say it for sure, but I remember the CompatHelper task never triggers the tests even when it performs a pull request to the master.

@codecov
Copy link

codecov bot commented Oct 24, 2023

Codecov Report

Merging #186 (02ae0ab) into master (5d8fdbe) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #186   +/-   ##
=======================================
  Coverage   92.32%   92.32%           
=======================================
  Files          40       40           
  Lines        2163     2163           
=======================================
  Hits         1997     1997           
  Misses        166      166           
Flag Coverage Δ
core 91.54% <ø> (ø)
komamri 93.88% <ø> (ø)
plots 93.80% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@beorostica beorostica merged commit 2b1114a into master Oct 25, 2023
16 checks passed
@beorostica beorostica deleted the compathelper/new_version/2023-10-21-00-33-39-343-03805923423 branch January 19, 2024 12:42
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