Skip to content

Conversation

TracerDS
Copy link
Contributor

No description provided.

@Dutchman101
Copy link
Member

Both you and Nico submitted a fixed PR at the exact same time (lol)

Code reviews and (self-)feedback should determine which of the 2 PR's is ideal, and to save us time, also please add details on what the problem was and how you fixed it; there's no specific "fix" commit.

Tracer's PR: #3530
Nico's PR: #3531

@Nico8340
Copy link
Member

This pull request does not restore the intended functionality of the original pull request, there are now two open and mine includes the rest.

@TracerDS
Copy link
Contributor Author

@Nico8340 You are the original owner of the PR. You may be the honoured one to fix it :)

@TracerDS TracerDS closed this Jun 30, 2024
@Nico8340
Copy link
Member

You are the original owner of the PR. You may be the honoured one to fix it :)

I'm okay with it, thanks.
I wrote the previous comment just fyi.

@TracerDS TracerDS deleted the 300624_resourceName branch July 1, 2024 07:48
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