conda-forge / mcstas-suite-feedstock

A conda-smithy repository for mcstas-suite.
BSD 3-Clause "New" or "Revised" License
2 stars 4 forks source link

FINAL Edits to mcstas-suite for build and execute on Windows via conda #14

Closed willend closed 8 months ago

willend commented 8 months ago

…a-forge/mcstas-suite-feedstock/pull/12 / https://github.com/conda-forge/mcstas-suite-feedstock/pull/13

Checklist

This PR allows building / testing on the Windows platform - now from my private fork to allow re-rendering... (It is yet another "clean slate" replacement of https://github.com/conda-forge/mcstas-suite-feedstock/pull/12 and https://github.com/conda-forge/mcstas-suite-feedstock/pull/13)

conda-forge-webservices[bot] commented 8 months ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

willend commented 8 months ago

@conda-forge-admin, please rerender

github-actions[bot] commented 8 months ago

Hi! This is the friendly automated conda-forge-webservice.

I tried to rerender for you but ran into some issues. Please check the output logs of the latest rerendering GitHub actions workflow run for errors. You can also ping conda-forge/core for further assistance or try re-rendering locally.

This message was generated by GitHub actions workflow run https://github.com/conda-forge/mcstas-suite-feedstock/actions/runs/7974808055.

willend commented 8 months ago

@conda-forge-admin, please rerender

github-actions[bot] commented 8 months ago

Hi! This is the friendly automated conda-forge-webservice.

I tried to rerender for you but ran into some issues. Please check the output logs of the latest rerendering GitHub actions workflow run for errors. You can also ping conda-forge/core for further assistance or try re-rendering locally.

This message was generated by GitHub actions workflow run https://github.com/conda-forge/mcstas-suite-feedstock/actions/runs/7974883267.

willend commented 8 months ago

@conda-forge-admin, please rerender

tkittel commented 8 months ago

Diff looks good to me, except the url of course.

willend commented 8 months ago

@tkittel suggested gameplan is:

  1. We see the tests run through correctly
  2. I create a new mcstas-3.4.9 tag on the main branch of the McCode repo
  3. I adapt this PR for the new URL/hash
  4. We merge
willend commented 8 months ago

Steps 1-3 done

willend commented 8 months ago

@tkittel "all systems go" it seems - merge?