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

Improves Chocolatey Package #68

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JPRuskin
Copy link

The Chocolatey package was rejected from the Chocolatey Community Repository after it's submission. This means that new versions were not allowed to be pushed (until that initial package is approved).

This commit makes changes to the nupkg files based on that feedback, and around instructions in the TODO and template(s).

  • We remove the TODO file, as that isn't really intended to be committed
  • We remove the README file, as that's a readme about the template and not this package
  • Updates to rendercli.nuspec(.template) based on feedback on the CCR review
  • Update the License.txt based on instructions / to move it on from being just a template
  • Update the Verification.txt to remove templated language

There may be further changes required, but this is a start towards getting the package approved, and the automated pipeline able to submit new versions.

The Chocolatey package was rejected from the Chocolatey Community Repository.

This commit makes changes to the nupkg files based on that feedback, and around instructions in the TODO and template(s).

- We remove the TODO file, as that isn't really intended to be committed
- We remove the README file, as that's a readme about the template and not this package
- Updates to rendercli.nuspec(.template) based on feedback on the CCR
- Update the License.txt based on instructions / to move it on from being just a template
- Update the Verification.txt to remove templated language

There may be further changes required, but this is a start.
@JPRuskin
Copy link
Author

@shawn-render I'm aware this isn't a high priority, but is there anything I can do to help out with this?

For what it's worth, though the package has been submitted it wasn't approved and so isn't very visible to folk.

(and sorry for pinging, I'm unsure who to talk to here as eropple has left)

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.

1 participant