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: add auto comment github action #58

Merged
merged 6 commits into from
Jan 20, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 28 additions & 0 deletions .github/workflows/auto-comment.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@
name: AutoComment
on: [issues, pull_request]
jobs:
run:
runs-on: ubuntu-latest
steps:
- uses: wow-actions/auto-comment@v1
with:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
issuesOpened: |
👋 @{{ author }}
Thank you for raising an issue. We will will investigate into the matter and get back to you as soon as possible.
Please make sure you have given us as much context as possible.
pullRequestOpened: |
⚠️ ⚠️ ⚠️

👋 @{{ author }}
Please remember that we use [**fast-forward merge**](https://git-scm.com/docs/git-merge#Documentation/git-merge.txt---ff-only) instead of using the usual *Create a merge commit*, *squash and merge* or *rebase and merge* for merging `develop` into `main` or vice versa.
You can execute it by creating a new comment in this PR with the following content:
```
fast-forward
```
***With an additional slash (`/`) before the command.***
**Please create this comment only after all tests have been completed positively!**

For merging your feature branch to `develop` please use [*squash and merge*](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/incorporating-changes-from-a-pull-request/about-pull-request-merges#squash-and-merge-your-commits).

Thanks! 🙏
28 changes: 0 additions & 28 deletions .github/workflows/pre-commit.yml

This file was deleted.

3 changes: 3 additions & 0 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,9 @@
default_language_version:
python: python3.10

ci:
skip: [poetry-lock, poetry-export]

repos:
- repo: https://github.com/compilerla/conventional-pre-commit
rev: v2.1.1
Expand Down
1 change: 1 addition & 0 deletions docs/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
***A collection of various pandas & scikit-learn compatible transformers for all kinds of preprocessing and feature engineering steps*** 🛠

[![testing](https://github.com/chrislemke/sk-transformers/actions/workflows/testing.yml/badge.svg?branch=main)](https://github.com/chrislemke/sk-transformers/actions/workflows/testing.yml)
[![Codacy Badge](https://app.codacy.com/project/badge/Grade/44093b8f6b28454fb8e0336ccb83cdc5)](https://www.codacy.com/gh/chrislemke/sk-transformers/dashboard?utm_source=github.com&utm_medium=referral&utm_content=chrislemke/sk-transformers&utm_campaign=Badge_Grade)
[![codecov](https://codecov.io/github/chrislemke/sk-transformers/branch/main/graph/badge.svg?token=LJLXQXX6M8)](https://codecov.io/github/chrislemke/sk-transformers)
[![deploy package](https://github.com/chrislemke/sk-transformers/actions/workflows/deploy-package.yml/badge.svg)](https://github.com/chrislemke/sk-transformers/actions/workflows/deploy-package.yml)
[![pypi](https://img.shields.io/pypi/v/sk-transformers)](https://pypi.org/project/sk-transformers/)
Expand Down
1 change: 0 additions & 1 deletion src/sk_transformers/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@ def check_ready_to_transform(
ValueError: If the input `X` does not contain the feature.
ValueError: if the input `X` does not contain all features.


Returns:
pandas.DataFrame: A checked copy of original dataframe.
"""
Expand Down