Sitelet https://web.archive.org/web/20260214155813/https://github.com/NativeScript/NativeScript/pull/8889
Skip to content

fix broken download of mk snapshot tools#8889

Closed
farfromrefug wants to merge 1 commit intoNativeScript:masterfrom
farfromrefug:patch-2
Closed

fix broken download of mk snapshot tools#8889
farfromrefug wants to merge 1 commit intoNativeScript:masterfrom
farfromrefug:patch-2

Conversation

@farfromrefug
Copy link
Collaborator

Part of a work to fix snaphshots for N 7

@NathanWalker
Copy link
Contributor

Closing as not pursuing snapshots further.

@farfromrefug
Copy link
Collaborator Author

Makes no sense not to fix this still when I give it to you... Total nonsense

@NathanaelA

This comment was marked as abuse.

@farfromrefug
Copy link
Collaborator Author

@NathanaelA I know you want to remove. And it is kind of funny your point about no one reacting to your issue about officially removing. No one knew about it.... Even me who is subscribed to all issues and PRs missed it. I don't think you can say it reflect the opinion of a community if you don't let the community knows...
It s about time your change your way of doing things and really include the community in your decision
I asked you already (many many times as you like to say) to create TSC discussion channels and other ways for the community to react and contribute to your TSC decisions. So please don't use that argument.

PS: as for the true matter of snapshot removal, I believe you are wrong to do it so fast without even having true number about the consequences (run time) , but I also am starting to believe it does indeed does not make a huge difference. It just makes me really sad to not have definite proof (which you should provide) before seeing this gone. We might create a bit hit on N (simply compare a react native cold runtime to N, makes you wonder if you should really use N)

@NathanaelA

This comment was marked as abuse.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants