Title: Remove psl-monad submodule · Issue #656 · PowerShell/PowerShell · GitHub
Open Graph Title: Remove psl-monad submodule · Issue #656 · PowerShell/PowerShell
X Title: Remove psl-monad submodule · Issue #656 · PowerShell/PowerShell
Description: Problem Having product code in the submodule of the main project makes the simplest fix a commit dance. The simple workflow is highly desirable to speed-up on-boarding. Having psl-monad submodule makes sd<->git integration easier. Meanwh...
Open Graph Description: Problem Having product code in the submodule of the main project makes the simplest fix a commit dance. The simple workflow is highly desirable to speed-up on-boarding. Having psl-monad submodule m...
X Description: Problem Having product code in the submodule of the main project makes the simplest fix a commit dance. The simple workflow is highly desirable to speed-up on-boarding. Having psl-monad submodule m...
Opengraph URL: https://github.com/PowerShell/PowerShell/issues/656
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Remove psl-monad submodule","articleBody":"## Problem\n\nHaving product code in the submodule of the main project makes the simplest fix a [commit dance](https://github.com/PowerShell/PowerShell/blob/master/docs/OperationalNotes.md#commit-dance).\nThe simple workflow is highly desirable to speed-up on-boarding.\n\nHaving `psl-monad` submodule makes sd\u003c-\u003egit integration easier.\nMeanwhile, it's possible to have a reasonable level of separationg and move psl-monad from submodule to a folder in the super-project.\n### Proposal\n1. Move `psl-monad\\monad` to root level `monad` folder.\n2. Move `psl-monad\\wmi` to root level `wmi` folder.\n\nThe reason behind it: make `git cherry-pick` from **gitsd admin** repo match one-to-one.\nThis way we keep sd -\u003e git story pretty clear. Question: can we keep simple cherry-pick workflow, but teach it to map root folder into `src\\monad`?\n- 2 branches would be still around in PowerShell/PowerShell: `master` and `source-depot`.\n- It's important to save git history.\n### Execution plan\n- We can achieve the desired result with a simple merge of two repos:\n\nFrom PowerShell\n\n```\ngit remote add monad github.com/PowerShell/psl-monad\ngit merge monad/master\ngit checkout -b source-depot\ngit merge monad/source-depot\n```\n\ncc @andschwa @zach-folwick @SteveL-MSFT @JamesWTruher \n\nUPD:\n### Inventarisation\n\nTree below shows the folder structure.\nNeed dig into some of the subtrees.\nNumber in brackets is the number of files.\n#### Legend\n\n| Mark | Meaning |\n| --- | --- |\n| :x: | no need to get anything from this folder |\n| :recycle: | need a subset from this folder |\n| :white_check_mark: | need everything from this folder |\n| :question: | investigate |\n#### Tree\n\nD:\\dev\\PowerShell\\src\\monad\n├──monad (26803)\n│ ├──core_powershell (3) :x:\n│ ├──incubation (392) :x: @lzybkr: but we should move a couple xsd files to miscfiles\n│ ├──InstallerSignJob (3) :x:\n│ ├──keys (9) :x:\n│ ├──loc (36) :question:\n│ ├──ManagedSignJob (3) :x:\n│ ├──miscfiles (124) :recycle: \n│ ├──nttargets (661) :x:\n│ ├──prototypes (2) :x:\n│ ├──Providers (83) :question: @lzybkr: maybe not needed\n│ ├──ScriptSignJob (13) :x:\n│ ├──Setup (972) :question: \n│ │\n│ ├──src (5403)\n│ │ ├──Activities (227) :white_check_mark: @lzybkr: we need it for workflow\n│ │ ├──build (3) :x:\n│ │ ├──cimSupport (41) :white_check_mark: \n│ │ ├──cmdletDesigner (440) :x:\n│ │ ├──commands (504) :white_check_mark: \n│ │ ├──CoreCLR (5) :white_check_mark: \n│ │ ├──DotNetCode (31) :question: \n│ │ ├──DscSupport (1) :white_check_mark: \n│ │ ├──engine (523) :white_check_mark: \n│ │ ├──graphicalhost (210) :x:\n│ │ ├──help (429) :recycle: \n│ │ ├──host (1136) :recycle: \n│ │ ├──LocalAccounts (30) :question: \n│ │ ├──logging (13) :white_check_mark: \n│ │ ├──m3p (168) :question: \n│ │ ├──minishell (20) :recycle: \n│ │ ├──namespaces (38) :question: \n│ │ ├──oneget (535) :question:\n│ │ ├──OSS (89) :x:\n│ │ ├──PSGetAddOn (2) :question: \n│ │ ├──PSReadline (37) :x:\n│ │ ├──samples (579) :question: \n│ │ ├──ScheduledJob (31) :question: \n│ │ ├──ScriptAnalyzer (174) :x:\n│ │ ├──sdk (12) :question: :\n│ │ ├──security (28) :white_check_mark: \n│ │ ├──singleshell (28) :white_check_mark: \n│ │ ├──tools (1) :x:\n│ │ └──utils (66) :white_check_mark: \n│ │\n│ ├──tests (15771)\n│ │ ├──ci (937) # binplaced\n│ │ │ ├──Appx (3) :x:\n│ │ │ ├──Lite3UnitTest (19) :x:\n│ │ │ ├──Lite_Template (19) :x:\n│ │ │ ├──MP (1) :x:\n│ │ │ ├──OnlineHelp (17) :question:\n│ │ │ ├──PowerShell (470) :recycle: \n│ │ │ ├──PowerShellMultiMachine (1) :x:\n│ │ │ ├──PowerShellPerformance (7) :question: \n│ │ │ ├──PowerShellStress (1) :x:\n│ │ │ ├──PSGet (39) :question: \n│ │ │ ├──PSOne (2) :question: \n│ │ │ ├──PSWF (56) :question: \n│ │ │ └──PSWS (300) :question:\n│ │ │\n│ │ ├──citargets (955) :recycle: # compiled\n│ │ ├──mae (3674) :recycle: \n│ │ ├──monad (10052) :recycle: \n│ │ ├──MultiMachine (148) :recycle:\n│ │ └──performance (4) :question: \n│ │\n│ ├──UnmanagedSignJob (3) :x:\n│ ├──V2 (263) :x:\n│ └──WTR (3056) :x:\n└──wmi (105) :white_check_mark:\n","author":{"url":"https://github.com/vors","@type":"Person","name":"vors"},"datePublished":"2016-03-12T00:40:37.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":9},"url":"https://github.com/656/PowerShell/issues/656"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:7e4cfa7f-1980-d15e-0146-ac5402780572 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | DBDC:152A69:14EC14:1B7F45:6A5AB2D5 |
| html-safe-nonce | bdf9501d54894f92d14f37ae9341aef74cd09d42e492bca24a2ebeacc25ac79f |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJEQkRDOjE1MkE2OToxNEVDMTQ6MUI3RjQ1OjZBNUFCMkQ1IiwidmlzaXRvcl9pZCI6IjY2ODcxMDM3NTkxMTcxMDM4MjkiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 0ac93b5e7e7c85bcc9d4c11bda1baf34f4205cd105e8ed1d4fa33ba81c20cb6b |
| hovercard-subject-tag | issue:140326640 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/PowerShell/PowerShell/656/issue_layout |
| twitter:image | https://opengraph.githubassets.com/72ef6b361d983ffeaf64322f33e97b824e4dd2687733d0239be5d56ef3eaa745/PowerShell/PowerShell/issues/656 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/72ef6b361d983ffeaf64322f33e97b824e4dd2687733d0239be5d56ef3eaa745/PowerShell/PowerShell/issues/656 |
| og:image:alt | Problem Having product code in the submodule of the main project makes the simplest fix a commit dance. The simple workflow is highly desirable to speed-up on-boarding. Having psl-monad submodule m... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | vors |
| hostname | github.com |
| expected-hostname | github.com |
| None | 21c53da6db0c40484841ba32fe19ecf485e7e734334faab487cd59d884d2bb33 |
| turbo-cache-control | no-preview |
| go-import | github.com/PowerShell/PowerShell git https://github.com/PowerShell/PowerShell.git |
| octolytics-dimension-user_id | 11524380 |
| octolytics-dimension-user_login | PowerShell |
| octolytics-dimension-repository_id | 49609581 |
| octolytics-dimension-repository_nwo | PowerShell/PowerShell |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 49609581 |
| octolytics-dimension-repository_network_root_nwo | PowerShell/PowerShell |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 6e85f20f54e1147d8162094880590864abefff6b |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width