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

add CLI #24

Merged
merged 4 commits into from
Jan 10, 2025
Merged

add CLI #24

merged 4 commits into from
Jan 10, 2025

Conversation

mgovers
Copy link
Member

@mgovers mgovers commented Jan 10, 2025

This PR adds an example CLI for the minimal VNF to PGM conversion introduced in #21

Test data in examples/convert_vnf_to_pgm/data

@mgovers mgovers added the feature New feature or request label Jan 10, 2025
@mgovers mgovers self-assigned this Jan 10, 2025
@mgovers mgovers added the do-not-merge This should not be merged label Jan 10, 2025
@mgovers mgovers force-pushed the feature/example-cli branch 2 times, most recently from c2eb31f to e14ec9b Compare January 10, 2025 09:28
Signed-off-by: Martijn Govers <[email protected]>
@mgovers mgovers force-pushed the feature/example-cli branch from e14ec9b to 8c2a6c3 Compare January 10, 2025 09:29
Signed-off-by: Martijn Govers <[email protected]>
@mgovers mgovers marked this pull request as ready for review January 10, 2025 11:10
@mgovers mgovers removed the do-not-merge This should not be merged label Jan 10, 2025
Base automatically changed from feature/minimal-conversion to main January 10, 2025 11:25
@mgovers mgovers enabled auto-merge January 10, 2025 11:46
@figueroa1395
Copy link
Contributor

Since only nitpicks remain, I will resolve the comments and approve (#24 (comment) and #24 (comment))

@mgovers mgovers added this pull request to the merge queue Jan 10, 2025
Merged via the queue into main with commit d1c0390 Jan 10, 2025
26 checks passed
@mgovers mgovers deleted the feature/example-cli branch January 10, 2025 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants