Skip to content
This repository has been archived by the owner on May 14, 2024. It is now read-only.

Update power-nukkitx 2.0 #2811

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

JblusItsMe
Copy link

No description provided.

@QuintenQVD0 QuintenQVD0 marked this pull request as draft March 20, 2024 07:09
@QuintenQVD0
Copy link
Collaborator

Their is more things wrong with this pr

This is not an update but a whole new egg

You literly removed the whole template

@JblusItsMe JblusItsMe marked this pull request as ready for review March 20, 2024 17:33
@ImKringle
Copy link
Contributor

This is again just a fully new egg

If making an update, only replace the lines you changed; Adding an entirely new egg is only if its getting its own section in the repo. This is an update, so only replace what is required. A whole new egg isnt required

Re:
`Their is more things wrong with this pr

This is not an update but a whole new egg

You literly removed the whole template`

Copy link
Contributor

@ImKringle ImKringle left a comment

Choose a reason for hiding this comment

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

Replace just whats changed. Install script, variables, etc - not upload a whole new file

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.

3 participants