Skip to content

Delete hide assignment in _adoptConfiguration*#197526

Merged
meganrogge merged 1 commit intomicrosoft:mainfrom
tisilent:cancel-dependency-hide
Nov 8, 2023
Merged

Delete hide assignment in _adoptConfiguration*#197526
meganrogge merged 1 commit intomicrosoft:mainfrom
tisilent:cancel-dependency-hide

Conversation

@tisilent
Copy link
Copy Markdown
Contributor

@tisilent tisilent commented Nov 6, 2023

fix #197390

@tisilent
Copy link
Copy Markdown
Contributor Author

tisilent commented Nov 6, 2023

@meganrogge hi .
The hide of dependent tasks should not be affected by other tasks.
I don't know if this will affect the design. 😄

@meganrogge
Copy link
Copy Markdown
Collaborator

@alexr00 looking at the code, I'm wondering if this was by design?

Copy link
Copy Markdown
Member

@alexr00 alexr00 left a comment

Choose a reason for hiding this comment

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

@meganrogge I think you added the hide property, but it seems ok to delete this.

Copy link
Copy Markdown
Member

@alexr00 alexr00 left a comment

Choose a reason for hiding this comment

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

@meganrogge I think you added the hide property, but it seems ok to delete this.

@meganrogge meganrogge enabled auto-merge November 7, 2023 14:53
@vscodenpa vscodenpa added this to the November 2023 milestone Nov 7, 2023
@meganrogge meganrogge merged commit 6785cae into microsoft:main Nov 8, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Dec 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Hidden Task Hiding Nonhidden Dependson Task

4 participants