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

ci: convert github issue templates from markdown to yaml #660

Merged
merged 2 commits into from
Mar 14, 2024

Conversation

juarezr
Copy link
Member

@juarezr juarezr commented Mar 14, 2024

This PR aims to improve the bug report and feature request processes.

Changes

  1. Converted github issue templates from markdown to yaml
  2. Categorized questions in bug and feature templates

Checklist

Use this checklist to ensure the quality of pull requests that include new code and changes to existing code.

  • Source Code guidelines:
    • Includes unit tests
    • New functions have docstrings with examples that can be run with doctest
    • New functions are included in API docs
    • Docstrings include notes for any changes to API or behavior
    • All changes are documented in docs/changes.rst
    • No user-facing changes
  • Versioning and history tracking guidelines:
    • Using atomic commits whenever possible
    • Commits are reversible whenever possible
    • There are no incomplete changes in the pull request
    • There is no accidental garbage added to the source code
  • Testing guidelines:
    • Tested locally using tox / pytest
    • Rebased to the master branch and tested before sending the PR
    • Automated testing passes (see CI)
    • Unit test coverage has not decreased (see Coveralls)
    • No source code changes
  • State of these changes is:
    • Just a proof of concept
    • Work in progress / Further changes needed
    • Ready to review
    • Ready to merge

@juarezr juarezr added Feature A nice to have thing that we don't have yet Maintainability Issues for code modernization, improve development, testing Build CI CD Issues affecting github pypi readthedocs coveralls conda-forge labels Mar 14, 2024
@juarezr juarezr self-assigned this Mar 14, 2024
@coveralls
Copy link

coveralls commented Mar 14, 2024

Pull Request Test Coverage Report for Build 8284919063

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 91.072%

Totals Coverage Status
Change from base Build 8282263331: 0.0%
Covered Lines: 13352
Relevant Lines: 14661

💛 - Coveralls

@juarezr juarezr merged commit 48e610d into petl-developers:master Mar 14, 2024
26 checks passed
@juarezr juarezr deleted the ci/issue-templates-yaml branch March 14, 2024 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build CI CD Issues affecting github pypi readthedocs coveralls conda-forge Feature A nice to have thing that we don't have yet Maintainability Issues for code modernization, improve development, testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants