Skip to content

chore: remove base/templates/bundles from rock.core#248

Merged
doudou merged 1 commit intomasterfrom
remove-base-templates-bundle
Feb 3, 2025
Merged

chore: remove base/templates/bundles from rock.core#248
doudou merged 1 commit intomasterfrom
remove-base-templates-bundle

Conversation

@doudou
Copy link
Copy Markdown
Member

@doudou doudou commented Jan 28, 2025

Depends on:

The rock-create-bundle now points to manual step-by-step from the rock&syskit documentation on how to create a bundle.

The template is obsolete, and difficult to keep in lockstep with the canonical way to create syskit app, that is syskit gen

Comment thread 04stable.autobuild Outdated
import_package 'base/templates/cmake_vizkit_widget'
import_package 'base/templates/vizkit3d_plugin'
import_package 'base/templates/bundle'
remove_fromm_default "base/templates/bundle"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
remove_fromm_default "base/templates/bundle"
remove_from_default "base/templates/bundle"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Shame ....

The rock-create-bundle now points to manual step-by-step from
the rock&syskit documentation on how to create a bundle.

The template is obsolete, and difficult to keep in lockstep with
the canonical way to create syskit app, that is `syskit gen`
@doudou doudou force-pushed the remove-base-templates-bundle branch from 996f790 to 3cfc57b Compare January 28, 2025 13:14
@doudou doudou requested a review from jhonasiv January 28, 2025 13:14
@doudou doudou merged commit f68cd94 into master Feb 3, 2025
@doudou doudou deleted the remove-base-templates-bundle branch February 3, 2025 17:15
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.

4 participants