Skip to content

Lists python multiexcel example#133

Merged
merschformann merged 1 commit intodevelopfrom
merschformann/list-multiexcel-app
Jul 23, 2025
Merged

Lists python multiexcel example#133
merschformann merged 1 commit intodevelopfrom
merschformann/list-multiexcel-app

Conversation

@merschformann
Copy link
Copy Markdown
Member

Description

Lists the python-ortools-multiknapsack-multiexcel app in the workflow config so that it can be cloned via Nextmv CLI.

Changes

  • Lists python-ortools-multiknapsack-multiexcel in workflow config.

@merschformann merschformann requested a review from Copilot July 23, 2025 22:18
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds the python-ortools-multiknapsack-multiexcel app to the workflow configuration to enable cloning via the Nextmv CLI.

  • Adds a new app entry for the Python OR-Tools multi-knapsack Excel example
Comments suppressed due to low confidence (1)

.nextmv/workflow-configuration.yml:169

  • [nitpick] The description should be consistent with the existing pattern. Consider changing to 'Use Python and OR-Tools - multi Excel knapsack.' to match the naming pattern of the previous entry which uses 'multi CSV knapsack'.
    description: Use Python and OR-Tools - multi knapsack via Excel.

Copy link
Copy Markdown
Member

@sebastian-quintero sebastian-quintero left a comment

Choose a reason for hiding this comment

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

LOL we missed it

@merschformann merschformann enabled auto-merge July 23, 2025 22:19
@merschformann merschformann merged commit 8d05a23 into develop Jul 23, 2025
40 checks passed
@merschformann merschformann deleted the merschformann/list-multiexcel-app branch July 23, 2025 22:20
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.

3 participants