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 workflow that will transition a POEM to integrated when a PR is merged #3079

Merged
merged 3 commits into from Dec 1, 2023

Conversation

swryan
Copy link
Contributor

@swryan swryan commented Nov 29, 2023

Summary

  • added the openmdao_update_poem workflow
    • this will run when a PR is opened or closed (and merged)
    • when a PR is opened, the Resolves tag will be checked to get the associated issue
    • the text of the issue will be checked for an associated POEM
    • If an Associated POEM is found:
      • if PR has just been opened, a message will be sent indicating that the POEM status will be transitioned to Integrated
      • if PR has just been merged, a workflow_dispatch event will be triggered on the POEMs repo to transition the POEM
    • Note that this relies on PRs and Issues using the prescribed GitHub templates

Also:

  • added an empty permissions dictionary by default for all workflows
  • updated NumPy, SciPy and pyOptSparse versions in the test workflow

Related Issues

Backwards incompatibilities

None

New Dependencies

None

@coveralls
Copy link

coveralls commented Nov 30, 2023

Coverage Status

coverage: 89.734% (-0.3%) from 90.021%
when pulling 4760e18 on swryan:3048_poem_integrated
into fee4510 on OpenMDAO:master.

@swryan swryan merged commit 5f62d2e into OpenMDAO:master Dec 1, 2023
8 checks passed
@swryan swryan changed the title Added a workflow that will check transition a POEM to integrated when a PR is merged Added a workflow that will check/transition a POEM to integrated when a PR is merged Dec 1, 2023
@swryan swryan changed the title Added a workflow that will check/transition a POEM to integrated when a PR is merged Added a workflow that will transition a POEM to integrated when a PR is merged Dec 1, 2023
@swryan swryan deleted the 3048_poem_integrated branch January 19, 2024 20:28
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.

Automate POEM status change to Integrated
3 participants