Skip to content

Roll forward lib/modules to get past state with broken normalization#1509

Open
Penguinwizzard wants to merge 1 commit into
microsoft:mainfrom
Penguinwizzard:user/penguinwizzard/roll_submodule
Open

Roll forward lib/modules to get past state with broken normalization#1509
Penguinwizzard wants to merge 1 commit into
microsoft:mainfrom
Penguinwizzard:user/penguinwizzard/roll_submodule

Conversation

@Penguinwizzard

Copy link
Copy Markdown
Member

Broken normalization in the lib/modules subrepo is causing enlistment issues in the Edge project; rolling forward the submodule to the commit 7bd8b516e2d93d1704834e0895733ae7bc2d1f43 picks up fixes made in that repo that amend this.

@bmehta001

bmehta001 commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Was the gitlink changed as intended in this PR? I don't see any changes.

Broken normalization in the lib/modules subrepo is causing enlistment
issues in the Edge project; rolling forward the submodule to the commit
7bd8b516e2d93d1704834e0895733ae7bc2d1f43 picks up fixes made in that
repo that amend this.
@Penguinwizzard
Penguinwizzard force-pushed the user/penguinwizzard/roll_submodule branch from bdc5700 to 34a1325 Compare July 20, 2026 17:23
@Penguinwizzard
Penguinwizzard requested a review from a team as a code owner July 20, 2026 17:23
@Penguinwizzard

Penguinwizzard commented Jul 20, 2026

Copy link
Copy Markdown
Member Author

Was the gitlink changed as intended in this PR? I don't see any changes.

Sorry, I missed adding it in the first iteration, because I had to manually construct the commit objects to get around codespaces' token scoping restrictions.

@dilijev dilijev left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM, thanks!

@bmehta001

Copy link
Copy Markdown
Contributor

Approved + re-ran the CIs that failed/timed out

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