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

Update power-nukkitx 2.0 - #2811

Open
JblusItsMe wants to merge 4 commits into
pelican-eggs:masterfrom
JblusItsMe:master
Open

JblusItsMe wants to merge 4 commits into
pelican-eggs:masterfrom
JblusItsMe:master

Conversation

@JblusItsMe

Copy link
Copy Markdown

No description provided.

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

Copy link
Copy Markdown
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
Copy Markdown
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`

@ImKringle ImKringle left a comment

Copy link
Copy Markdown
Contributor

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