Skip to content

fix: fixing the number of connections counter to be correctly named a… - #610

Closed
kvassall-unity wants to merge 2 commits into
developfrom
feature/profiler_connections_fix
Closed

fix: fixing the number of connections counter to be correctly named a…#610
kvassall-unity wants to merge 2 commits into
developfrom
feature/profiler_connections_fix

Conversation

@kvassall-unity

Copy link
Copy Markdown
Contributor

…nd function properly

@Rosme

Rosme commented Mar 15, 2021

Copy link
Copy Markdown

Please make sure you target release/0.1.0

@kvassall-unity

Copy link
Copy Markdown
Contributor Author

Please make sure you target release/0.1.0

There are two PRS

See:

#613

@Rosme

Rosme commented Mar 15, 2021

Copy link
Copy Markdown

I don't think you should make two PRs. I believe release will be merge into develop at some point. @MFatihMAR @mattwalsh-unity can you confirm?

@0xFA11

0xFA11 commented Mar 15, 2021

Copy link
Copy Markdown
Contributor

we should target release/0.1.0 and later we will merge release/0.1.0 periodically back into develop.
if we had this PR merged into develop, that'd give us conflict issues. if this will be (or already is) in release/0.1.0, I'd say we don't need this PR to be merged into develop at this point.

@kvassall-unity

Copy link
Copy Markdown
Contributor Author

This pr was older, approved and already set to develop. We can close this if we want. But the release pr is the more important one.

@kvassall-unity

Copy link
Copy Markdown
Contributor Author

Closing since the release now has this

@0xFA11
0xFA11 deleted the feature/profiler_connections_fix branch March 15, 2021 18:58
@0xFA11
0xFA11 restored the feature/profiler_connections_fix branch March 15, 2021 18:58
@0xFA11
0xFA11 deleted the feature/profiler_connections_fix branch March 15, 2021 18:58
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.

4 participants