Skip to content

Debug test on Linux - #16010

Merged
0101 merged 5 commits into
dotnet:mainfrom
0101:fix-test
Feb 5, 2024
Merged

Debug test on Linux#16010
0101 merged 5 commits into
dotnet:mainfrom
0101:fix-test

Conversation

@0101

@01010101 commented Sep 20, 2023

Copy link
Copy Markdown
Contributor

These are the symbol uses I see:

[|TestProject_88c93ff2, Binding,(1,7--1,27);
generic parameter a, UseInType,(2,15--2,17);
generic parameter a, UseInType,(2,31--2,33);
generic parameter a, UseInType,(2,31--2,33); TFirst, Binding,(2,21--2,27);
TFirstV_1`1, Binding,(2,5--2,14);valx,Binding,(3,6--3,7);
TFirst, Use,(4,4--4,10);valx,Use,(4,11--4,12);valf,Binding,(3,4--3,5);valx,Binding,(5,7--5,8);valop_Addition,Use,(5,13--5,14);valx,Use,(5,11--5,12);valf2,Binding,(5,4--5,6);ModuleFirst,Binding,(1,28--1,39);TestProject_88c93ff2,Open,(1,7--1,27);generic parameter a,UseInType,(2,16--2,18);generic parameter a,UseInType,(2,33--2,35);
generic parameter a, UseInType,(2,33--2,35); TSecond, Binding,(2,22--2,29);
TSecondV_1`1, Binding,(2,5--2,15);valx,Binding,(3,6--3,7);
ModuleFirst, Use,(4,4--4,15);valf,Use,(4,4--4,17);valx,Use,(4,18--4,19);TSecond,Use,(5,4--5,11);valx,Use,(5,12--5,13);valf,Binding,(3,4--3,5);valx,Binding,(6,7--6,8);valop_Addition,Use,(6,13--6,14);valx,Use,(6,11--6,12);valf2,Binding,(6,4--6,6);ModuleSecond,Binding,(1,28--1,40);TestProject_88c93ff2,Open,(1,7--1,27);generic parameter a,UseInType,(2,15--2,17);generic parameter a,UseInType,(2,31--2,33);
generic parameter a, UseInType,(2,31--2,33); TThird, Binding,(2,21--2,27);
TThirdV_1`1, Binding,(2,5--2,14);valx,Binding,(3,6--3,7);
ModuleFirst, Use,(4,4--4,15);valf,Use,(4,4--4,17);valx,Use,(4,18--4,19);TThird,Use,(5,4--5,10);valx,Use,(5,11--5,12);valf,Binding,(3,4--3,5);valx,Binding,(6,7--6,8);valop_Addition,Use,(6,13--6,14);valx,Use,(6,11--6,12);valf2,Binding,(6,4--6,6);ModuleThird,Binding,(1,28--1,39);TestProject_88c93ff2,Open,(1,7--1,27);generic parameter a,UseInType,(2,14--2,16);generic parameter a,UseInType,(2,29--2,31);
generic parameter a, UseInType,(2,29--2,31); TLast, Binding,(2,20--2,25);
TLastV_1`1, Binding,(2,5--2,13);valx,Binding,(3,6--3,7);
ModuleSecond, Use,(4,4--4,16);valf,Use,(4,4--4,18);valx,Use,(4,19--4,20);ModuleThird,Use,(5,4--5,15);valf,Use,(5,4--5,17);valx,Use,(5,18--5,19);TLast,Use,(6,4--6,9);valx,Use,(6,10--6,11);valf,Binding,(3,4--3,5);valx,Binding,(7,7--7,8);valop_Addition,Use,(7,13--7,14);valx,Use,(7,11--7,12);valf2,Binding,(7,4--7,6);EntryPointAttribute,UseInAttribute,(9,2--9,12);EntryPointAttribute,UseInType,(9,2--9,12);member .ctor,Use,(9,2--9,12);valop_PipeRight,Use,(11,7--11,9);valf,Use,(11,3--11,4);valignore,Use,(11,10--11,16);valprintfn,Use,(12,3--12,10);valmain,Binding,(10,4--10,8);ModuleLast,Binding,(1,28--1,38)|]

@0101
0101 requested a review from a team as a code ownerSeptember 20, 2023 13:42
@0101

0101 commented Sep 20, 2023

Copy link
Copy Markdown
ContributorAuthor

Well, it passed now 🤷‍♂️

@vzarytovskii

Copy link
Copy Markdown
Member

I don't get why is it flaky and sometimes shows different amount of results.

@0101

0101 commented Nov 1, 2023

Copy link
Copy Markdown
ContributorAuthor

I don't get why is it flaky and sometimes shows different amount of results.

Is it still? I think we should just merge this so we can at least see what's missing (or extra) when it happens.

@vzarytovskii

Copy link
Copy Markdown
Member

I don't get why is it flaky and sometimes shows different amount of results.

Is it still? I think we should just merge this so we can at least see what's missing (or extra) when it happens.

Surprisingly it just blow ups out of the blue sometimes. And then it's ok for two weeks...

Comment threadtests/FSharp.Compiler.ComponentTests/FSharpChecker/CommonWorkflows.fs Outdated
T-Groand others added 2 commits November 24, 2023 10:26
@vzarytovskii

Copy link
Copy Markdown
Member

Closing it, feel free to reopen

@0101

0101 commented Feb 2, 2024

Copy link
Copy Markdown
ContributorAuthor

I still think we should merge this to see what happens when the test fails.

@01010101 reopened this Feb 2, 2024
@github-actions

Copy link
Copy Markdown
Contributor

✅ No release notes required

@0101
0101 merged commit 3ac064e into dotnet:mainFeb 5, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants

@0101@vzarytovskii@psfinaki@T-Gro