Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Guidelines and code to reflect new procedures #1754

Open
7 of 16 tasks
AdmiringWorm opened this issue Nov 23, 2021 · 0 comments
Open
7 of 16 tasks

Update Guidelines and code to reflect new procedures #1754

AdmiringWorm opened this issue Nov 23, 2021 · 0 comments
Assignees

Comments

@AdmiringWorm
Copy link
Member

AdmiringWorm commented Nov 23, 2021

In the coming times, there will be changes to how code and packages can be contributed to this repository, as such the guidelines will need to be updated to reflect these new changes.

We currently have a very loose definition of what necessary steps are needed before contributing any new features, bugs, or migrations of new packages but these will be enforced in a more strict way going forward once the Guidelines are updated.

Among the changes that are planned, we have:

  • Update current issue templates to use issue forms, that will be required to be filled out. ((#1754) Add issue forms to use #1788)
  • Update the guidelines to require a discussion and approval being created before submitting any pull requests for new features, or migrations of packages (pull requests can be submitted for bugs or typos without opening an issue/discussion). ((#1754, #1940) Update documentation about contributing to repo #2155)
  • Enforce that each package needs to have at least 1 CODEOWNER associated with them, if there is a missing code owner or the maintainer listed as code owner is unresponsive/unwilling the package can end up being removed from the repository.
  • Pull request template needs to be updated to include the steps any testing have been done, and what issues are fixed in the repository (testing need to be enforced by the Reviewer).
  • Add GitHub action to reference any user that is listed as a CODEOWNER when necessary on pull requests.
  • Add GitHub action to do basic linting of all changed files (possibly through the Super-Linter GitHub Action.
  • Make it clear in the contribution documentation that pre-releases are provided as is, and is a low priority.
  • Move any feature/enhancement requests not already being worked on to discussions (for those that have not been agreed upon that they need/should be implemented).
  • Update guidelines to disallow any changes to .github or scripts directory without approval from a Repository maintainer.
  • Update guidelines to mention avoid the use of globbing patterns, unless absolutely necessary. The use of these has led to the same bug surfacing over and over again.
  • Add development container files to the repository to have a common ground that can be used.
  • Migrate from using the stale bot for issues and pull requests to a GitHub Action instead. (#1754) Replace Stalebot with GitHub action #1798
  • Add GitHub action to validate some minimal pieces of data (both metadata and PowerShell scripts) that packages should adhere to.
  • Create a common set of tags to be used as categorization for packages #1940
  • Lock down repository to require Pull Request reviews, especially from code owners
  • Create GitHub Action to allow people with write access to the repository to push fix versions of packages

There are several reason why these changes are needed going forward, and to summarize some of them:

  • Over the years there has been an influx of requests to enhance packages with additional features, but they have either been unclear how what would be expected or none was willing to pick up the work necessary to make the changes.
  • There are often bug reports with missing details, which often results in the reports being closed or never responded to.
  • There is the uncertainty of what is accepted and not accepted between contributors and maintainers alike.
  • There are currently no fixed procedures that we follow when reviewing any incoming pull request, which leads to certain things being missed.
  • There are changes being made to scripts (both package scripts, and repository scripts), that causes either incompatible changes or changes to the behaviour or intention of the script.
  • The lockdown is being done due to changes being made that are either not appropriate for the repository, or for the Chocolatey Community Repository. As such requiring a review from a person either listed as a code owner, or from a person with write access to the repository is necessary.
@AdmiringWorm AdmiringWorm self-assigned this Nov 24, 2021
@AdmiringWorm AdmiringWorm pinned this issue Jan 17, 2022
AdmiringWorm added a commit to AdmiringWorm/chocolatey-coreteampackages that referenced this issue Jan 17, 2022
This commit adds new issue forms that need
to be filled out, as well as adding links to the discussion
page for things we do not want to have issues created directly for.
AdmiringWorm added a commit to AdmiringWorm/chocolatey-coreteampackages that referenced this issue Jan 18, 2022
Co-authored-by: Paul Broadwith <[email protected]>
Co-authored-by: Gary Ewan Park <[email protected]>
AdmiringWorm added a commit to AdmiringWorm/chocolatey-coreteampackages that referenced this issue Jan 18, 2022
Co-authored-by: Paul Broadwith <[email protected]>
Co-authored-by: Gary Ewan Park <[email protected]>
Co-authored-by: TheCakeIsNaOH <[email protected]>
AdmiringWorm added a commit to AdmiringWorm/chocolatey-coreteampackages that referenced this issue Feb 14, 2022
This commit adds new issue forms that need
to be filled out, as well as adding links to the discussion
page for things we do not want to have issues created directly for.
AdmiringWorm added a commit to AdmiringWorm/chocolatey-coreteampackages that referenced this issue Feb 14, 2022
Co-authored-by: Paul Broadwith <[email protected]>
Co-authored-by: Gary Ewan Park <[email protected]>
Co-authored-by: TheCakeIsNaOH <[email protected]>
AdmiringWorm added a commit to AdmiringWorm/chocolatey-coreteampackages that referenced this issue Feb 14, 2022
…mments

Co-authored-by: Paul Broadwith <[email protected]>
Co-authored-by: Gary Ewan Park <[email protected]>
Co-authored-by: TheCakeIsNaOH <[email protected]>
AdmiringWorm added a commit to AdmiringWorm/chocolatey-coreteampackages that referenced this issue Feb 14, 2022
When we renamed the repository previously the stalebot
stopped working on this repository, this PR re-adds the
stalebot but as part of a daily GitHub action instead.
AdmiringWorm added a commit to AdmiringWorm/chocolatey-coreteampackages that referenced this issue Feb 14, 2022
When we renamed the repository previously the stalebot
stopped working on this repository, this PR re-adds the
stalebot but as part of a daily GitHub action instead.
AdmiringWorm added a commit to AdmiringWorm/chocolatey-coreteampackages that referenced this issue Feb 14, 2022
AdmiringWorm added a commit to AdmiringWorm/chocolatey-coreteampackages that referenced this issue Feb 14, 2022
Co-authored-by: Paul Broadwith <[email protected]>
Co-authored-by: Gary Ewan Park <[email protected]>
AdmiringWorm added a commit to AdmiringWorm/chocolatey-coreteampackages that referenced this issue Feb 14, 2022
@TheCakeIsNaOH TheCakeIsNaOH unpinned this issue Feb 28, 2022
@TheCakeIsNaOH TheCakeIsNaOH pinned this issue Feb 28, 2022
AdmiringWorm added a commit to AdmiringWorm/chocolatey-coreteampackages that referenced this issue Feb 24, 2023
…o repo

This commit updates the documentation we have for contributing
to the repository that needs to be followed.
AdmiringWorm added a commit to AdmiringWorm/chocolatey-coreteampackages that referenced this issue Feb 24, 2023
…o repo

This commit updates the documentation we have for contributing
to the repository that needs to be followed.
AdmiringWorm added a commit to AdmiringWorm/chocolatey-coreteampackages that referenced this issue Feb 24, 2023
…o repo

This commit updates the documentation we have for contributing
to the repository that needs to be followed.
AdmiringWorm added a commit to AdmiringWorm/chocolatey-coreteampackages that referenced this issue Feb 24, 2023
…o repo

This commit updates the documentation we have for contributing
to the repository that needs to be followed.
AdmiringWorm added a commit to AdmiringWorm/chocolatey-coreteampackages that referenced this issue Mar 28, 2023
…o repo

This commit updates the documentation we have for contributing
to the repository that needs to be followed.
AdmiringWorm added a commit to AdmiringWorm/chocolatey-coreteampackages that referenced this issue Oct 19, 2023
…o repo

This commit updates the documentation we have for contributing
to the repository that needs to be followed.
corbob added a commit that referenced this issue Oct 19, 2023
…-code-to-reflect-new-procedures

(#1754, #1940) Update documentation about contributing to repo
pauby pushed a commit to pauby/chocolatey-coreteampackages that referenced this issue Mar 10, 2024
…o repo

This commit updates the documentation we have for contributing
to the repository that needs to be followed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant